CovidTrackerFr / vitemadose-front

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

Correction de l'utilisation des mauvais fichiers de cache pour l'auto-complétion #211

Closed Luwangel closed 3 years ago

Luwangel commented 3 years ago

Les fichiers de cache d'auto-complétion ont été renommés sur la PR https://github.com/CovidTrackerFr/vitemadose-front/pull/149 car certains ne passaient pas sur Windows.

Le dernier refacto en date a supprimé l'appel aux fichiers renommés : https://github.com/CovidTrackerFr/vitemadose-front/commit/d0dc4632cfa371fe2dcf18570f7b12fa17b37ae0#diff-5f7f40c0d5c6024f626185600d5905793cd993fb722a6159875052c956078653.

Todo

fcamblor commented 3 years ago

@Luwangel il y a un truc que je ne m'explique pas... je vois bien la régression introduite sur d0dc463, mais pourtant, lorsque je regarde la prod VMD, je vois des autocompletes qui partent sur des vmd_*.json alors que la MEP d'hier soir aurait dû embarquer cette régression, non ?

Pour moi, il est important de se poser la question afin de savoir si on donne un petit sursis aux "vieux" fichiers d'autocomplete, dans le cas où ces derniers auraient pu avoir été utilisé/mis en cache par une vieille version en prod.

Dans le cas contraire, je suis complètement d'avis de m'en séparer pour éviter les erreurs futures :-)

Luwangel commented 3 years ago

@fcamblor Et bien figure toi que je viens de comprendre pourquoi. J'ai regardé les commits d'hier soir, et @Floby a fait une modif sur master :D https://github.com/CovidTrackerFr/vitemadose-front/commit/4dc698f87907a34d6ce7cd6359269138c9c79d64

fcamblor commented 3 years ago

ok cool, ben let's go alors (le temps qu'on MEP, les gens auront switché sur la nouvelle version en vmd_*.json

(je laisse la PR encore un peu ouverte au cas où d'autres contributeurs souhaitent redire quelque chose)

Floby commented 3 years ago

yup j'ai pris la modif au passage la dernière fois :) je ferme

Luwangel commented 3 years ago

@Floby La PR supprimait surtout les vieux fichiers :( Tu ne veux pas qu'on la garde ?

Mise à jour : on ne garde que la suppression des anciens fichiers