linkclump icon indicating copy to clipboard operation
linkclump copied to clipboard

Manifest V3

Open marcozink opened this issue 1 year ago • 1 comments

Not sure if this will work but I've updated the manifest to V3 and added a minor version increase to the extension.

The main purpose is to prevent the extension for stop being supported on future versions of Google Chrome.

marcozink avatar Aug 22 '24 05:08 marcozink

Did you test this?

benblack86 avatar Aug 22 '24 12:08 benblack86

I've taken a small look at the changes, and I can report that they need some extra work.

image

If you fix these errors, you get even more errors. First, it is about messaging, and then, it is about local storage, and that's where I stopped.

Uncaught ReferenceError: localStorage is not defined
Uncaught TypeError: Cannot read properties of undefined (reading 'addListener')
chrome.extension.onMessage.addListener(function(request, sender, callback) {

Also, with the new manifest, you can only give one background file, so if you still want to split the files, you will need a build step.

I don't think I can fix this at the moment, but when Google kicks this extension from the app store, I'll be back.

wvanderp avatar Sep 02 '24 08:09 wvanderp

I believe this PR got pretty close, but a few things weren't working https://github.com/benblack86/linkclump/pull/221

benblack86 avatar Sep 06 '24 17:09 benblack86