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

Refactor extension to use MV3 in firefox and uniformize the codebase #37

Merged
merged 4 commits into from
Mar 8, 2023

Conversation

LuisDuarte1
Copy link
Member

@LuisDuarte1 LuisDuarte1 commented Mar 2, 2023

This makes the code base more stable across firefox and chrome, changing the watch from a websockets server to a https server that the extension polls.

Also adds a permissions request/welcome page to the extension, because only firefox makes host_permissions optional, violating the standard haha.

@LuisDuarte1 LuisDuarte1 changed the title Refactor extension to use MV3 in firefox and uniformize the code base Refactor extension to use MV3 in firefox and uniformize the codebase Mar 2, 2023
@thePeras thePeras mentioned this pull request Mar 3, 2023
Copy link
Member

@ttoino ttoino left a comment

Choose a reason for hiding this comment

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

Apart from the comment I left and some aesthetic stuff I don't really agree with (mostly the use of var), LGTM!

README.md Show resolved Hide resolved
@LuisDuarte1 LuisDuarte1 merged commit 539c335 into main Mar 8, 2023
@LuisDuarte1 LuisDuarte1 deleted the refactor/firefox-mv3 branch March 8, 2023 18:09
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.

2 participants