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
@nachtjasmin nachtjasmin force-pushed the 14-transition-to-manifest-v3 branch from 4363df0 to d091a02 Compare June 14, 2023 15:04
@nachtjasmin nachtjasmin force-pushed the 14-transition-to-manifest-v3 branch from 709ce3b to fd5e780 Compare June 21, 2023 13:39
@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 force-pushed the 14-transition-to-manifest-v3 branch from cd1ddfc to d102a5e Compare July 12, 2023 20:31
@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
@nachtjasmin nachtjasmin force-pushed the 14-transition-to-manifest-v3 branch 2 times, most recently from 61b6a32 to b4e06d7 Compare July 17, 2023 12:01
@nachtjasmin nachtjasmin force-pushed the 14-transition-to-manifest-v3 branch from 6804ef4 to 371781b Compare July 28, 2023 19:55
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