MTES-MCT / histologe

Histologe - Signaler un problème de mal-logement et accélérer sa prise en charge
https://histologe.beta.gouv.fr
MIT License
5 stars 0 forks source link

[BO - zones] Ajouter des zones à exclure sur le partenaire + UX #3320

Open numew opened 2 days ago

numew commented 2 days ago

Ticket

3287

Description

Ajout de la feature des zone à exclure d'un partenaire avec refonte partielle de la partie partenaire

Changements apportés

Pré-requis

make execute-migration name=Version20241119085440 direction=up make npm-build

Tests

emilschn commented 1 day ago

Tests :

numew commented 1 day ago

Tests :

  • J'ai voulu ajouter une zone juste en copier-coller, mais il a l'air de bloquer si il n'y a pas de fichier.
  • Dans la continuité : si il y a une erreur de validation de formulaire, le bouton Valider reste disabled
  • Dans la fiche d'une zone, on ne voit pas les partenaires exclus, c'est volontaire ?

Les point un et deux étaient liés, c'est corrigé. Bien vu pour le troisième c'était un oubli

emilschn commented 1 day ago

Autres retours de test :

numew commented 1 day ago

Autres retours de test :

  • c'est pas directement lié à cette PR, mais dans la fiche d'une zone, ce serait bien de mettre des gouttières entre la carte et le texte, non ? (fr-grid-row--gutters)
  • si on édite un utilisateur dans un partenaire, au retour, idéalement, ce serait cool d'être redirigé vers le bon onglet (on a fait un truc similaire dans la fiche signalement)

Ok pour les deux points, c'est corrigé

sonarcloud[bot] commented 12 hours ago

Quality Gate Failed Quality Gate failed

Failed conditions
24.0% Duplication on New Code (required ≤ 3%)
B Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE