Skip to content

Retours FTHD: Listes vides

Sylvain Boulade requested to merge fthd-feedback-liste-vide into main

Contexte

Nous traitons dans cette MR un retour de FTHD du ticket https://datahub.incubateur.tech/infrastructure/indicateurs/-/issues/160, spécifiquement:

Pour les opérateurs : laisser la mention sur les opérateurs de la ZTD vide lorsqu’il n’y a pas de ZTD peut porter à confusion. On pourrait l’enlever.

En effet on peut voir que l'indicateur "Opérateurs en zone très tense" (je me demande d'ailleurs s'il n'y a pas une typo ici tense -> dense, à corriger peut-être ultérieurement) par exemple pour la région Ile-de-France est vide quand on clique dessus.

Ca peut potentiellement être le cas pour d'autres indicateurs.

Contenu

Ce qui se passe c'est que nous pouvons avoir des listes contenant une chaine de caractère vide. Pour remédier à cela on peut filtrer la liste avant de décider si nous voulons afficher ou non le widget.

Comme je rajoute quelques tests unitaires, j'en ai profité également pour réorganiser le dossier test afin qu'il soit calqué sur le dossier src, à voir si ça vous semble pertinent sinon je peux revert ce changement sans problème.

J'ajoute quelques petites notes en commentaire également.

Screenshots

Avant Après

A noter que l'indicateur est aussi masqué en version PDF mais je n'ai pas encore pu faire tourner le proxy-pdf en local donc je n'ai pas pu prendre de screenshot pour le moment.

Edited by Sylvain Boulade

Merge request reports