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

Manifest v3 #11

Merged
merged 13 commits into from
Aug 24, 2022
Merged

Manifest v3 #11

merged 13 commits into from
Aug 24, 2022

Conversation

JohnyP36
Copy link
Owner

Any additions, changes or removals is at the Authors discretion.
You're free to counterargue (to a certain point) if you disagree with the decision.
To avoid being banned, don't constantly re-open or create new (related) issue reports or pull requests.

Prerequisites

  • I've read the Code of Conduct and the Contributing Guidelines
  • This is more than one or two issue(s)
  • I performed a cursory search of the issue tracker to avoid opening a duplicate issue
    • Your issue may already be reported.
  • I tried to reproduce the issue when...
    • Other webextensions related to Youtube are disabled
    • using a new, unmodified browser profile
  • I am running the latest version of the extension

Describe the issue

I updated the extension to manifest version 3, which Google will implement in January 2023.

Screenshot(s)

Screenshot(s)

Browser & Extension version

Browser name & version Extension Version Operating System
`MS Edge 104` `1.6.5` `Windows 10, 21H2`

Notes

Still not working properly are the loop function and the autoplay in YouTube Music

Solved the same issue as BPower0036 had in Nomes77/YT-Nonstop#8
In manifest v2 `chrome.runtime.onInstalled.addListener` was enough to make the popup disable on every domain except YouTube. 
In version 3 I had to add `onStartup` so it will not only disable the popup on installation, but also after restarting your browser.
`function loadPageAccess()` didn't work (the script didn't get loaded. 
In order to create a unity I constructed the whole script in this way using `var`.
Forgot to change those => should be the same as the word after `var`.
@JohnyP36 JohnyP36 linked an issue Aug 24, 2022 that may be closed by this pull request
5 tasks
@JohnyP36 JohnyP36 merged commit a44e036 into main Aug 24, 2022
@JohnyP36 JohnyP36 deleted the manifest_v3 branch August 24, 2022 15:47
@JohnyP36 JohnyP36 restored the manifest_v3 branch August 24, 2022 15:48
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.

Migration to Manifest v3
1 participant