Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Manifest V3 Extension #88

Merged
merged 1 commit into from
May 21, 2024
Merged

Manifest V3 Extension #88

merged 1 commit into from
May 21, 2024

Conversation

joeytroy
Copy link
Contributor

@joeytroy joeytroy commented May 3, 2024

Moving to version 3 of the manifest. Also pushing the version of our application to 1.0.0 and getting rid of the x.x.x.x versioning.

Moving to version 3 of the manifest. Also pushing the version of our application to 1.0.0 and getting rid of the x.x.x.x versioning.
@joeytroy
Copy link
Contributor Author

joeytroy commented May 3, 2024

@jonrandy, @deepfriedfilth, @beepidibop,

If you could look over this commit to make sure it looks OK. I have verified it locally but want to make sure it looks good before we move to the new V3 Extension and it still works properly with Firefox which I have not tested. I updated it based on the Googles update to Manifest V3 https://developer.chrome.com/docs/extensions/develop/migrate along with updating the .js as well to support the newer chrome.storage.local over the older localStorage. If it looks good I will go ahead and merge it in.

@deepfriedfilth
Copy link
Contributor

deepfriedfilth commented May 3, 2024

@joeytroy can't load the extension with manifest v3 changes in firefox as is, unfortunately. background.service_worker flag would need to be enabled in about:config, which isn't ideal, although it appears you can disable "persistent" key in "background" node of manifest, but unsure if that's the ultimate solution. (nope, firefox still wouldn't install without re-adding "scripts": ["src/background.js"] in addition to service_worker entry, and Chrome web store will most likely reject it for having both, according to some users in the GH issue linked below)

Some relevant reading:

@joeytroy
Copy link
Contributor Author

joeytroy commented May 3, 2024

@deepfriedfilth thanks for looking into that we may need to create a new branch just for Firefox users.

@jonrandy would you be willing to keep up with a Firefox branch since you uploaded the newest version of the extension? Since Google is forcing us to manifest 3 not sure we are going to have a choice unless we can figure out how to implement Firefox and Chrome compatibility into the manifest 3.

@jonrandy
Copy link
Contributor

jonrandy commented May 4, 2024

So, if I'm reading correctly, both can use manifest v3 but Firefox requires an extra line in the manifest that will cause it to be rejected in Chrome store?

@joeytroy
Copy link
Contributor Author

joeytroy commented May 4, 2024

@jonrandy that sounds correct I have not yet looked at the shared links that @deepfriedfilth posted.

@jonrandy
Copy link
Contributor

Just had a quick read through those links and this seems to be the quickest solution

@joeytroy joeytroy merged commit 01ece61 into master May 21, 2024
@joeytroy joeytroy deleted the manifest_v3 branch May 21, 2024 15:11
@deepfriedfilth
Copy link
Contributor

deepfriedfilth commented May 21, 2024

@jonrandy (unrelated) when you have a moment, could you upload the latest "release" (v.0.5.9.31) via Mozilla Add-Ons?

It terms of this manifest v3 cross-browser compatibility snafu, I've seen some solutions that employ build tooling to output the appropriate manifest or other differences for browser-specific versions.. unsure about the Chrome Web Store, but I believe Mozilla will make you jump through more hoops in verification if we adapt things to utilize task runner/automation solutions (I know they specifically mention webpack when uploading an add-on to their portal, but any pre-processing might likely apply)

@joeytroy
Copy link
Contributor Author

@deepfriedfilth @jonrandy so I merged the code and have submitted the code to the Chrome store and we are now at version 1.0.0 to start us fresh with updates. We can create a Fork called Firefox or add a new repository for Firefox since Manifest 3 is causing issues. Let me know how to proceed with that. Also, not sure how much time it's worth spending on Firefox as we only have 11 Firefox users compared to 720 users in the Chrome Store. I leave that decision up to you all.

@jonrandy
Copy link
Contributor

Am kinda super busy with work and a bunch of other projects, so I think the best plan sounds like waiting on this to see if there is any demand for further fixes.

I've pretty much abandoned the GMail interface in favour of Shortwave for personal mail, so that user count is likely down to 10

@deepfriedfilth
Copy link
Contributor

deepfriedfilth commented Jun 10, 2024

@jonrandy thanks for your contributions to the project over the years, but I totally get moving on to that new new.

I know you're keeping busy but would you mind adding me as an author via Mozilla Add-Ons so I could at least upload the latest compatible version (v0.5.9.31) for the few fellow Firefox users? (Email is nkawatski@gmail.com 🤞🏻) I've been submitting an unlisted version myself but would love to make my latest chat related fixes available to anyone who may be interested.

I wouldn't mind diving in and seeing how to move forward using manifest v3 and maintaining a FF version! @joeytroy I know you mentioned forking a FF version, but perhaps a 'firefox' branch of this repo might be sufficient?

@jonrandy
Copy link
Contributor

@deepfriedfilth Done!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants