-
-
Notifications
You must be signed in to change notification settings - Fork 221
Upgrade to MV3 (WIP) #373
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
base: master
Are you sure you want to change the base?
Upgrade to MV3 (WIP) #373
Conversation
|
Le build PR semble ne pas fonctionner à cause du module web-ext utilisé pour le lint : 2 solutions possibles pour valider les runs pour le MV3 :
|
|
Je serais plutôt pour garder warningsAsErrors: true et ignorer juste ce warning là. Et peut-être remonter le problème à web-ext eux-mêmes ? Si c'est le seul moyen de faire une extension compatible avec ff et chrome, il ne faudrait pas que ça casse le linter. |
|
En fait je ne comprends pas: quel que soit le navigateur, ce n'est pas valide d'avoir un |
Le support inter-navigateur est très particulier, comme souvent lorsque le dev de la fonctionnalité (ici MV3) est faite par un seul acteur sans standard global du W3C, les différences sont détaillées ici : https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/background#browser_support Ici c'est la prop |
|
Hello, Pour rappel, une des fonctionnalités à tester suite à la migration est celle de l'auth via referer, comme avec les provider suivants : Si vous avez également des identifiants BNF, je peux également les tester étant donné qu'il y a des constantes particulières sur certains services. |
|
Hello @kvnco, je me souviens que tu m'avais prêté ton accès MaBM pour fixer un problème très spécifique avec MaBM. Je me demandais s'il tu étais prêt à prêter tes identifiants à @mxgnr pour qu'il puisse vérifier si son intégration pour le Manifest V3 fonctionne correctement. Nous n'avons malheureusement pas de meilleur solution pour tester ces intégrations. Ceci permettrait d'être compatible avec le navigateur Google Chrome. @mxgnr, Si Kevin répond positivement, pourrais-tu lui fournir une adresse de contact ? Merci à vous deux ! |
|
Sans réponse, je serais d'avis que dans tous les cas on passe en MV3. L'extension n'est pas disponible sur Chrome alors que c'est le navigateur le plus utilisé au monde. Je trouve que cela serait plus pertinent pour étendre l'accès d'usage de cette extension dans tous les cas, et ce, même si MaBM et consort ne fonctionnent pas à la release de la MV3. Est-ce qu'autrement, tout est ok pour un build firefox & chrome pour toi @mxgnr ? Merci ! |
Hello, je vais rebase ma branche avec celle par défaut histoire d'être à jour et refaire quelques tests. Je ne suis pas contre l'idée de publier ce PoC mv3, en revanche je recommanderai de bumper la release en version majeure (3.0) et de la publier en Bêta tant que le système d'auth en referrer n'a pas été pleinement testé et éprouvé, dans un premier temps sur le Chrome Web Store. On peut également publier une release 3.0 Bêta pour Firefox mais étant donné que le store Firefox add-ons n'accepte pas la distribution de versions Bêta, elle ne sera disponible que sur ce repo, à installer manuellement. Qu'en pensez-vous ? |
WIP -> #236
Adaptation de l'extension pour la compatibilité ManifestV3 (MV3) suite à la dépréciation de la MV2 par le navigateur Google Chrome.