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

Transition to Manifest v3 #15

Open
wants to merge 15 commits into
base: main
Choose a base branch
from
Open

Conversation

nachtjasmin
Copy link
Contributor

@nachtjasmin nachtjasmin commented May 25, 2023

Description

Closes #14. This PR depends on #32 to be merged.

This PR migrates the extension to Manifest v3. Although the current manifest version (2) still works fine, we can expect that Google is pushing the new manifest version even more.
For us, this has only one big advantage: We don't need to request the permission to access all pages.

This might be seen as disadvantage, because it requires one click more, but personally, I love this.
It ensures that our extension cannot mess with other websites on it's own. The confirmation by the click, that this is really a Mastodon page, is huge.

TODO

  • Migrate pronouns.page support (we need to request the permissions manually)
  • Add button to permanently enable protoots on this page

@nachtjasmin nachtjasmin added the enhancement New feature or request label May 25, 2023
@nachtjasmin nachtjasmin added this to the Chrome Release milestone May 25, 2023
@nachtjasmin nachtjasmin self-assigned this May 25, 2023
@nachtjasmin nachtjasmin linked an issue May 25, 2023 that may be closed by this pull request
@nachtjasmin nachtjasmin force-pushed the 14-transition-to-manifest-v3 branch 2 times, most recently from faf83bf to 4363df0 Compare June 2, 2023 21:12
@nachtjasmin nachtjasmin requested a review from ItsVipra June 2, 2023 21:18
@nachtjasmin nachtjasmin marked this pull request as ready for review June 2, 2023 21:19
src/options/options.js Outdated Show resolved Hide resolved
@nachtjasmin nachtjasmin force-pushed the 14-transition-to-manifest-v3 branch from fd5e780 to 555d897 Compare July 1, 2023 13:49
@nachtjasmin nachtjasmin marked this pull request as draft July 1, 2023 13:51
@nachtjasmin nachtjasmin force-pushed the 14-transition-to-manifest-v3 branch 6 times, most recently from 5022542 to 4a689b8 Compare July 5, 2023 14:11
@nachtjasmin nachtjasmin force-pushed the 14-transition-to-manifest-v3 branch 4 times, most recently from 1d1aecc to 3ce6502 Compare July 7, 2023 23:56
@nachtjasmin nachtjasmin marked this pull request as ready for review July 7, 2023 23:56
@nachtjasmin nachtjasmin requested a review from ItsVipra July 7, 2023 23:56
@nachtjasmin
Copy link
Contributor Author

Note to self: fix scripts on windows.

@nachtjasmin nachtjasmin force-pushed the 14-transition-to-manifest-v3 branch 2 times, most recently from fc6e5ad to cd1ddfc Compare July 12, 2023 19:45
@nachtjasmin nachtjasmin marked this pull request as draft July 12, 2023 20:21
@nachtjasmin nachtjasmin marked this pull request as ready for review July 12, 2023 20:58
@nachtjasmin nachtjasmin force-pushed the 14-transition-to-manifest-v3 branch 4 times, most recently from 00256cb to e89e62a Compare July 17, 2023 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Transition to Manifest v3
2 participants