Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unités – Backoffice d'édition des unités et administrations #2373

Closed
AdelineCelier opened this issue Jul 20, 2023 · 5 comments · Fixed by MTES-MCT/monitorenv#509
Closed
Assignees

Comments

@AdelineCelier
Copy link
Collaborator

AdelineCelier commented Jul 20, 2023

La plupart des informations concernant les unités seront éditables directement depuis la fiche unité dans Monitor.

Le backoffice doit seulement servir à ajouter/modifier une administration, ou ajouter/modifier/archiver une unité.

Onglet 1 – Tableau administrations

  • 1 colonne avec le nom de l'administration
  • 1 colonne avec les noms des unités rattachées à l'administration
  • en bout de ligne, 1 bouton "modifier l'administration" qui ouvre un formulaire avec le nom de l'administration
  • au-dessus du tableau, 1 bouton "ajouter une administration" qui ouvre le même formulaire mais vide
  • au-dessus du tableau, une barre de recherche qui permet de chercher dans les 2 colonnes (donc une unité ou une admin.)

Onglet 2 – Tableau unités actives

  • 1 colonne avec le nom de l'unité
  • 1 colonne avec le nom de l'administration à laquelle elle est rattachée
  • en bout de ligne
    • 1 bouton "modifier" qui ouvre un formulaire avec le nom de l'unité et un menu déroulant avec les admin.
    • 1 bouton "archiver" qui ouvre une popup de confirmation puis qui bascule l'unité dans le tableau "unités archivées"
  • au-dessus du tableau, 1 bouton "ajouter une unité" qui ouvre le même formulaire que "modifier" mais vide
  • au-dessus du tableau, une barre de recherche qui permet de chercher dans les 2 colonnes (donc une unité ou une admin.)

Onglet 3 – Tableau unités archivées

  • 1 colonne avec le nom de l'unité
  • 1 colonne avec le nom de l'administration à laquelle elle était rattachée
  • 1 colonne avec la date et l'heure d'archivage de l'unité
  • en bout de ligne, un bouton "réactiver l'unité"
@AdelineCelier
Copy link
Collaborator Author

AdelineCelier commented Sep 26, 2023

Revue du backoffice

Question des administrations archivées

  • Rajouter l'archivage d'une administration :
    • si l'administration n'a plus aucune unité active, elle peut être archivée avec juste une pop-up de confirmation disant "Confirmez-vous l'archivage de cette administration ? Elle n'apparaîtra plus dans MonitorEnv, elle ne sera plus utilisée que pour les statistiques
    • si l'administration a des unités actives, une pop-up apparaît avec un message disant "Cette administration est rattachée à des missions ou des signalements. Veuillez l'en détacher avant de la supprimer ou bien l'archiver."
  • Rajouter soit un filtre soit un onglet pour consulter séparément les administrations actives et celles archivées.

Question des unités archivées

  • Rajouter soit un filtre soit un onglet pour consulter séparément les administrations actives et celles archivées.
  • Si facile à faire, rajouter un filtre "administration" ? (pas prio)
  • Rajouter la suppression des unités :
    • si une unité n'est rattachée à aucune instance, elle est supprimée avec juste une pop-up de confirmation ("confirmez-vous la suppression de cette unité ? Ceci entraînera la suppression de toutes ses informations (moyens, contacts...)
    • si une unité est rattachée à une mission ou un signalement, une pop-up apparaît avec un message disant "Cette unité est rattachée à des missions ou des signalements. Veuillez l'en détacher avant de la supprimer ou bien l'archiver."
  • Rajouter l'archivage des unités. Mettre une pop-up de confirmation disant "Confirmez-vous l'archivage de cette unité ? Elle n'apparaîtra plus dans MonitorEnv, elle ne sera plus utilisée que pour les statistiques".

Autres points

  • Titre des sections : remplacer "administration de..." par "gestion de..." : "Gestion des administrations", "Gestion des unités"...

@AdelineCelier
Copy link
Collaborator Author

AdelineCelier commented Oct 16, 2023

Revue n°2 du backoffice

Points en plus des points précédents qui n'ont pas encore été traités (cf. cases non cochées dans la première revue)

Revue UX

  • L'archivage d'une administration n'est pas bloqué quand elle a encore des unités actives 👍
  • ajout d'un département : rajouter de la recherche dans le champ. Est-ce qu'il serait possible de pouvoir les chercher aussi par numéro ? 👍
  • et aussi sur les départements : @VincentAntoine est-ce qu'il y aurait un moyen de leur assigner un département automatiquement ? Ou bien les utilisateurs devront le faire à la main ? Comment l'utilisateur choisit-il le département d'une unité (par ex d'un patrouilleur) ? Par rapport à sa base ?
  • Les mises à jour d'infos ne sont effectives qu'au rechargement de la page. 👍
  • La suppression d'une unité n'est pas bloquée quand elle est citée dans une mission : est-ce que c'est un bug ou est-ce que ça n'est pas vérifiable en inté ? 👍

Revue UI

  • Faire le menu de droite comme sur le backoffice de Fish (en l'état il est vraiment peu lisible) : fond Charcoal, texte Gainsboro, avec de vrais entrées bien espacées 👍
  • Revoir les marges dans les pop-up d'avertissement : il faut qu'il y ait 40 px de marge à gauche et à droite. Soit mettre les textes à la ligne en ne laissant pas de petits mots en bout de ligne, soit élargir la pop-up. 👍

@AdelineCelier AdelineCelier reopened this Oct 16, 2023
ivangabriele added a commit to MTES-MCT/monitorenv that referenced this issue Oct 17, 2023
ivangabriele added a commit to MTES-MCT/monitorenv that referenced this issue Oct 18, 2023
## Related Pull Requests & Issues

- #699
- MTES-MCT/monitorfish#2373

----

- [x] Tests E2E (Cypress)
@AdelineCelier
Copy link
Collaborator Author

AdelineCelier commented Oct 19, 2023

Revue n°3 du backoffice

Revue UX

  • J'ai testé de créer une unité puis de la supprimer : la création marche bien, en revanche la suppression ne fonctionne pas. Je clique sur "supprimer" dans la pop-up mais l'unité reste dans la liste. Après quand je veux l'éditer, il y a bien une phrase qui dit "cette unité n'existe pas ou n'existe plus". Mais faudrait la virer de la liste non ? Et je me dis que ce serait pas mal de mettre un toast vert disant "l'unité xx a bien été supprimée", pour que l'utilisateur voit que son action a fonctionné. A noter que ça ne le fait pas pour les administrations : quand j'en supprime une elle disparaît bien de la liste.
    => ce problème n'est pas réapparu dans les derniers tests, on le met de côté tant qu'on ne le voit pas revenir.
  • Je viens aussi de penser qu'il ne faudrait garder dans la liste que les départements côtiers, je doute qu'une unité ait un jour pour département l'Ain ou la Corrèze ;)
  • J'ai l'impression qu'il y a un problème dans la barre de recherche : j'ai désarchivé l'unité "Adroit (historique)" de la Marine Nationale, elle est bien passée du tableau des unités archivées à celui des unités actives, mais quand je tape "adroit" dans la barre de recherche, rien ne remonte. C'est une question de temps d'actualisation ?
  • Changer le wording des pop-up de confirmation d'archivage : sous la bonne suggestion de Loup, mettre "elle ne sera utilisée que pour les statistiques" 👍

Revue UI

  • Il y a l'air d'y avoir encore des cas où les marges gauche et droite des pop-up ne font pas 40 px...c'est galère à gérer ces marges ? 👍
    image

@AdelineCelier
Copy link
Collaborator Author

@ivangabriele

  • j'ai recréé un ticket spécifique pour le fix sur la barre de recherche.
  • concernant la liste des départements, vu qu'elle est recherchable maintenant et qu'il y a les numéros, pas besoin dans un premier temps de virer les dpts non littoraux.
  • enfin, comme noté ci-dessus, tant que le bug sur la màj de la liste après suppression ne réapparaît pas je propose qu'on le laisse de côté.

Je ferme donc ce ticket !

@AdelineCelier
Copy link
Collaborator Author

Update, le bug de màj de la liste est réapparu, j'ai donc créé le ticket 929 sur le sujet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants