search

CovidTrackerFr / vitemadose-front

Interface utilisateur de l'outil ViteMaDose
https://vitemadose.covidtracker.fr
Other
80 stars 51 forks source link

#180 Fix problème d'affichage de la page des centres #171

Closed nhumblot closed 3 years ago

nhumblot commented 3 years ago

Retour arrière partiel sur le lazy loading des vues. Le bundling de leaflet ne semble pas géré correctement par vitejs et la page des centres ne s'affiche plus en production.

L'objectif de cette PR est de rétablir la fonctionnalité.

Je n'ai à ce stade pas encore trouvé comment permettre un fonctionnement en lazy loadant le module. Je continue d'investiguer pour apporter une solution de long terme qui clôturera #138 .

fcamblor commented 3 years ago

Je pense avoir identifié le soucis : on a des imports (lazy) dans la page chronodose qui viennent interférer dans la génération des modules vite

En fait, je pense que lorsque le même module (en l'occurence, leaflet) est chargé en lazy sur 2 modules différents, uniquement l'un des deux fichiers minifié et généré

Je me demande s'il ne s'agit pas d'un bug dans vitejs 🤔 (en tous cas, on a un workaround car on a la "chance" de n'utiliser leaflet que sur une seul page ... j'ai juste eu à supprimer la dépendance de la page chronodose car de toutes manière elle était inutile)

fcamblor commented 3 years ago

PS: visible ici : https://dev.vitemado.se/revert-lazy-loading-of-centers/

nhumblot commented 3 years ago

J'ai ouvert un ticket sur le projet vite avec tous les éléments pour reproduire le problème. Il est étrange d'avoir un fonctionnement correct en dev mais pas une fois packagé en mode prod.

fcamblor commented 3 years ago

Super cool Nico d'avoir remonté le problème chez ViteJS 👌

Je suis intrigué d'avoir le fin mot de l'histoire.