MTES-MCT / monitorenv

🛥️🏝️🌊 Suivi et appui au contrôle de l'environnement marin 🛥️🏝️🌊
https://beta.gouv.fr/startups/monitorfish.html
MIT License
6 stars 1 forks source link

Unités – Retouches diverses sur le backoffice et la fiche unité #926

Closed AdelineCelier closed 11 months ago

AdelineCelier commented 12 months ago

Types de contact dans la fiche unité

Revoir le wording dans les pop-ups d'alerte

Retouches ergonomie/UI

ivangabriele commented 11 months ago

Vu ensemble pour celui-ci :

Aujourd'hui, on peut direct taper un nom de contact personnalisé. Il faudrait empêcher ça, sinon le risque c'est que les utilisateurs fassent toujours ça et ne choisissent pas des items de la liste.

C'est donc un bug si c'est le cas et je ne le reproduis pas sur ma branche de fix.

AdelineCelier commented 11 months ago

Types de contacts fiche unité

AdelineCelier commented 11 months ago

@ivangabriele je remets ce ticket en todo pour la question de l'ordre alphabétique

ivangabriele commented 11 months ago

le bug que je t'ai remonté sur le fait qu'on peut éditer un contact même s'il n'est pas personnalisé est toujours présent https://github.com/MTES-MCT/monitorenv/assets/71878648/0d51b208-6705-4505-ac66-8cece96c349a

@AdelineCelier Le nom dans la base de donnée est bien un nom personnalisé. Je pense qu'il y a eu un soucis de nommage sur les unités dont le nom est à renseigner lors de la migration. Cette valeur devrait être à "UNKNOWN" qui elle correspond au nom prédifini "Nom de contact non renseigné" et là elle est directement égale à "Nom du contact à renseigner".

Je pourrais éventuellement ajouter une migration qui remplace tous les "Nom du contact à renseigner" par "UNKNOWN" si on veut que le frontend détecte ça comme un nom prédifini. Mais c'est un peu dirty de faire ça.

@VincentAntoine ?

AdelineCelier commented 11 months ago

La liste est bien en ordre alphabétique maintenant, et je vois bien les ajouts/changements de valeurs dont on a parlé Ivan. Il reste à faire la partie avec Vincent. Je remets donc ce ticket en todo en attendant, pour qu'on ne le perde pas de vue.

AdelineCelier commented 9 months ago

Ce ticket semble réglé, on le ferme.