-
Notifications
You must be signed in to change notification settings - Fork 31
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
MV3 Support? #28
Comments
At the moment, this will not work with MV3. However, we plan to release this functionality in the near future. |
Any news guys? |
The end of life of V2 reaching in a year and google started to slow down reviews of V2 starting this month.... |
@rushilsrivastava Do you need any help implementing the V3 Support? |
Hey guys, thanks for being patient. Let me address a couple of things real quick:
Yup, MV2 end-of-life (no new updates will be allowed to existing MV2 extensions) is slated for January 2023. I predict that Google will probably end up delaying this, but nevertheless definitely pushes the need for V3 support. While I couldn't find any announcement for slow release of V2 extensions (anecdotally, all our reviews have been approved within 24-hours), as of January 2022, no new extensions with MV2 will be allowed. All this information can be found on Google's Support Page.
Here's my roadmap:
As far as I know, there shouldn't be much work to get this to work in V3 other than reworking how background listeners work. I have no specific timeline to when I will be able to start work on V3, but if others are able to contribute PRs, I can commit to reviewing them and accelerating the roadmap accordingly. |
There seems to be a fix for it already, maybe you can use this https://github.com/rubenspgcavalcante/webpack-extension-reloader/pull/139/files @rushilsrivastava? It's described more here: rubenspgcavalcante/webpack-extension-reloader#125 |
Hey guys, I just submitted a PR to fix this issue. My setup is MacOS Chrome + MV3. Both the extension and background page are working fine with this PR, leaving the content script untested. Feel free to use my fork before the PR merged. |
I been using reorx fork for now, any plans to merge it in? |
come on |
I can confirm that PR created by lmichelin in #319 works for MV3 extensions. |
I intend to merge one of the open PRs by the end of November. There seems to be open review comments/concerns on each one of these PRs at the moment |
There are two ways to resole the problem.
|
I adapted @sherifmayika 's advice and finally made my fork work. Please try https://github.com/reorx/webpack-ext-reloader, which should be fully compatible with MV3. |
is it published to NPM? |
Yes it's npm i -D @reorx/webpack-ext-reloader
…On Tue, Feb 28, 2023 at 4:24 PM Bnaya Peretz ***@***.***> wrote:
I adapted @sherifmayika <https://github.com/sherifmayika> 's advice and
finally made my fork work. Please try
https://github.com/reorx/webpack-ext-reloader, which should be fully
compatible with MV3.
is it published to NPM?
—
Reply to this email directly, view it on GitHub
<#28 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AADDDVEZKFVIAP2OCH64G3DWZWY5FANCNFSM5AH4IDJA>
.
You are receiving this because you commented.Message ID:
***@***.***>
--
Sincerely,
Xiao Meng
reorx.com
|
@reorx thanks for taking the initiative! Your fork doesn't seem to work for me though, getting an error: Reproduction: https://github.com/just-jeb/angular-chrome-extension/tree/mv3-background-reload
|
Can you try the instruction at
https://github.com/reorx/webpack-ext-reloader please?
The fork may not working properly in the development environment.
…On Tue, Mar 14, 2023 at 4:14 PM JeB ***@***.***> wrote:
@reorx <https://github.com/reorx> thanks for taking the initiative! Your
fork doesn't seem to work for me though, getting an error:
[image: image]
<https://user-images.githubusercontent.com/9823087/224936678-733873bd-9560-4203-bf89-5eeee7bf6553.png>
.
Reproduction:
https://github.com/just-jeb/angular-chrome-extension/tree/mv3-background-reload
1. git clone
2. yarn
3. yarn watch
4. Load as unpacked from the dist/angular-chrome-extension folder
—
Reply to this email directly, view it on GitHub
<#28 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AADDDVA6EQ7M363BVMXQRP3W4ASFVANCNFSM5AH4IDJA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
--
Sincerely,
Xiao Meng
reorx.com
|
If by instruction you mean readme then it does follow the instruction. |
-(function(_window) { +(function() { |
Or (function(_window) {
let window: any
if (_window?.document !== undefined) {
window = _window
} I guess it does the same. |
try instead of guess |
Thank you for such a useful comment. |
did you test? |
Any updates regarding manifest v3 support? |
@xadamxk https://github.com/reorx/webpack-ext-reloader is working properly in my projects, you can have a try |
Yup, I got your fork working after commenting here - thank you. |
Any hints on how to resolve the "Uncaught TypeError: Cannot read properties of undefined (reading 'document')" issue? @reorx |
It's broken and is not being maintained, use this fork instead: https://github.com/reorx/webpack-ext-reloader |
There are 3 open PRs for Manifest V3, each has there own issues. I'll see if I can clean them up soon, but this repo is being maintained for V2. |
#386 has MV3 support and will be included in the next release (1.1.10). |
It comes from the code that is generated by |
We're using this in a MV3 service worker without issue (aside from a separate library causing issues in the SW), which is why your comment doesn't make sense to me. I will look further into this today. |
To all who are concerned about this project, I have updated @reorx/webpack-ext-reloader to version 1.6.0, which addresses a persistent issue where the plugin continually raised errors, significantly disrupting the development experience. The errors in question were primarily the following:
Overall, Additional quality-of-life improvements include:
|
I would like to, but I'm afraid I don't have time to achieve this at the
moment. I'm busy with my own projects, and these changes are being made
while I am actively developing my extensions. Since the code has changed
significantly from the original version, it requires a lot of effort to
create a proper PR. Maybe I can get to it sometime later.
…On Jan 4, 2024 at 16:08:32, Rushil Srivastava ***@***.***> wrote:
@reorx <https://github.com/reorx> Can you open a PR with your changes?
CC @vialoh <https://github.com/vialoh>
—
Reply to this email directly, view it on GitHub
<#28 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AADDDVGQTEDWFVVOCAQ7NYLYMZPQBAVCNFSM5AH4IDJKU5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TCOBXGY3DONRXGAZA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
It will support MV3?
thanks.
The text was updated successfully, but these errors were encountered: