(PC-33007) chore(web): remove service worker #7274
Conversation
There was a problem hiding this comment.
il me semble qu'on avait déjà tenté de supprimer le service worker et qu'on avait du des soucis lors du déploiement / mise à jour en prod, si mes souvenirs sont bons, on avait du rollback pour corriger rapidement
il faudra etre particulièrement vigilant lors de la mise en prod que coté web tout ce passe bien
public/index.html
Outdated
| @@ -111,7 +111,6 @@ | |||
| media="(device-width: 1024px) and (device-height: 1366px) and (-webkit-device-pixel-ratio: 2)" | |||
| rel="apple-touch-startup-image" | |||
| /> | |||
| <link rel="manifest" href="<%- PUBLIC_URL %>/manifest.json" /> | |||
There was a problem hiding this comment.
c'est dommage, on ne pourra plus l'installer la version web sur l'écran d'accueil
est-ce qu'il y a un moyen facile de supprimer le service worker tout en gardant le manifest.json ?
There was a problem hiding this comment.
T'avais reussi à faire marche l'installation de la version web sur l'écran accueil? Perso j'y suis jamais arrivé donc je sais pas si ça marchait
There was a problem hiding this comment.
je viens de le faire à l'instant avec Firefox sur Android
There was a problem hiding this comment.
J'ai retiré cette ligne parceque j'avais cette erreur dans un build local:
Access to manifest at 'https://passculture.app/manifest.json' from origin 'http://localhost:4173' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.
Mais peut être que je peux laisser la ligne?
There was a problem hiding this comment.
Après on est d'accord que le manifest n'apporte pas grand chose si on a plus le SW?
There was a problem hiding this comment.
de ce que j'en avais compris, avoir le manifest permet d'avoir le bouton permettant d'installer la web app sur l'écran d'accueil
le service worker, permet de gérer le cache des requetes (et donc l'offline)
il y a des PWA installable qui ne fonctionnent que si on est connecté, je ne sais pas si elles ont un service worker (j'imagine que non)
There was a problem hiding this comment.
Je confirme qu'on peut avoir le manifest sans pour autant installer le service worker, qui lui doit être lancé via le JS.
Au pire, il est peut-être plus simple dans un premier temps de faire des appels "vides" des fonctions qui existent et qui sont appelées par le SW ?
There was a problem hiding this comment.
C'est bizarre, je n'arrive plus a reproduire l'erreur CORS que j'avais en build local. Je restaure le manifest.json du coup
cgerrard-pass
force-pushed
the
PC-33007-remove-service-worker
branch
3 times, most recently
from
9334373
to
59816d8
Compare
cgerrard-pass
force-pushed
the
PC-33007-remove-service-worker
branch
from
59816d8
to
3a6a7bd
Compare
|
|
Link to JIRA ticket: https://passculture.atlassian.net/browse/PC-33007
Flakiness
If I had to re-run tests in the CI due to flakiness, I add the incident on Notion
Checklist
I have:
Screenshots
delete if no UI change
Best Practices
Click to expand
!when you know that the value can’t benullorundefined).Test specific:
usertofireEvent.