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

Update README.md #906

Merged

Conversation

Jaifroid
Copy link
Member

This textual PR (no code) closes #905 by updating the base Firefox version to >=52, and also modifying some other minor details in the readme.

Additionally, the new default status of Service Worker mode is added to the readme.

@Jaifroid Jaifroid self-assigned this Oct 24, 2022
@Jaifroid
Copy link
Member Author

Jaifroid commented Oct 24, 2022

I have one query for @mossroy. We currently list Firefox OS as a supported platform, which is true in the sense that we test on it (and I aim to continue testing on the simulator). But I wonder if it is time to move it to the "deprecated" section, given that the app in Firefox OS only runs in a now-deprecated mode (JQuery). I don't feel strongly about this, but it would probably be logical.

@Jaifroid Jaifroid added this to the v3.6 milestone Oct 24, 2022
@mossroy
Copy link
Contributor

mossroy commented Oct 24, 2022

Sure, you can clearly move it to "deprecated".

@Jaifroid Jaifroid merged commit fa944fe into master Oct 26, 2022
@Jaifroid Jaifroid deleted the Update-readme-for-compatibility-and-content-injection-mode branch October 26, 2022 12:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Extension no longer works in Firefox 45 ESR - seems to be CSP issue
2 participants