Skip to content

Migration to mv3#78

Closed
Jonghakseo wants to merge 7 commits intoN1ck:masterfrom
Jonghakseo:feat/mv3
Closed

Migration to mv3#78
Jonghakseo wants to merge 7 commits intoN1ck:masterfrom
Jonghakseo:feat/mv3

Conversation

@Jonghakseo
Copy link

@Jonghakseo Jonghakseo commented Aug 3, 2024

@N1ck
Copy link
Owner

N1ck commented Aug 29, 2024

Hey @Jonghakseo thanks for taking a look at this, sorry for being so slow to reply to this. Just wondering if you've tested whether this also works in Firefox? Hopefully I have more time soon I can do a more thorough test!

@Jonghakseo
Copy link
Author

Jonghakseo commented Oct 2, 2024

@N1ck

I added support for firefox.

Added for different support for mv3 itself, which requires a small modification to the manifest after the build.

I've tested it myself, please check it out! And please take a look at the other PRs related to broken UI.

"$schema": "https://json.schemastore.org/chrome-manifest",
"manifest_version": 3,
"name": "GIFs for GitHub",
"version": "1.2.11",
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This probably needs to be bumped up... Please be careful when deploying.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@N1ck N1ck mentioned this pull request Mar 22, 2025
@N1ck
Copy link
Owner

N1ck commented Mar 22, 2025

Thank you! Have included this in #83

@N1ck N1ck closed this Mar 22, 2025
@Jonghakseo Jonghakseo deleted the feat/mv3 branch March 23, 2025 00:14
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