search

leonarf / VCA4D

Code source du site web crée par le Basic pour mettre en valeur les résultats des études de VCA4D
1 stars 2 forks source link

Popup de comparaison vide #75

Closed maximepiard closed 1 month ago

maximepiard commented 1 month ago

Contexte

Lorsqu'on ouvre la popup de comparaison, la liste des etudes ne s'affiche pas

image

maximepiard commented 1 month ago

@leonarf Je pense que ça vient du changement que tu as fait ici, https://github.com/leonarf/VCA4D/commit/f33b2a0671eb623a4a8f5e25a1ed07cf9d83945f

A cette ligne studyData.id est toujours mango-guinea-bissau (cette string est dans le json) et donc on ne trouve pas l'étude dans la liste de data.json avec getStudy.

A mon avis, il faudrait éviter de changer les noms / données des fichiers importés à la main, et toujours de baser sur le script d'import pour éviter ce genre de souci. On pourrait donc soit réimporter l'étude en changeant les noms dans les xls, soit seulement changer le prettyName dans data.json

Qu'en penses tu ? Je peux te laisser régler ce bug ?

leonarf commented 1 month ago

Merci pour la détection du bug. c'est corrigé.