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

Signalements – Ajouter un résumé à l'historique des signalements et revoir la manière d'empiler les signalements similaires #3188

Closed
3 tasks done
AdelineCelier opened this issue May 14, 2024 · 3 comments · Fixed by #3635, #3644 or #3688
Labels
dev feat. enhancement feature enhancement

Comments

@AdelineCelier
Copy link
Collaborator

AdelineCelier commented May 14, 2024

  • Ajouter un résumé en tête de l'onglet "Historique des signalements" (cf. cet écran)
  • Regrouper les alertes similaires dans l'historique comme c'est fait dans les signalements actifs 👍🏻
  • Dans les deux onglets, ajouter un moyen de voir les différentes dates des signalements empilés (cf. écrans 21 et 22)
@AdelineCelier
Copy link
Collaborator Author

AdelineCelier commented Sep 12, 2024

@louptheron quelques retours :

  • pour moi il n'est pas nécessaire de mettre la pastille en haut à droite de la carte du signalement ou de l'observation s'il n'y en a qu'un seul. Elles n'y sont pas dans l'onglet "signalements en cours", je pense qu'on peut faire de même dans les signalements archivés. 👍🏻

Image

  • dans le résumé des signalements archivés, pour les signalements manuels, je me demande s'il ne faut pas faire figurer le n° du NATINF en plus de son intitulé – mais c'est aussi vrai que ça coupe un peu la lecture. Je te laisse me dire ce que tu en penses. 👍🏻

  • enfin, dans l'historique des signalements archivés, je ne pense pas qu'il soit nécessaire de mentionner 2 fois la dernière alerte d'une pile de plusieurs alertes. Cf. dans la capture ci-dessous, la 2è alerte et la dernière sont les mêmes. 👍🏻

Image

louptheron added a commit that referenced this issue Sep 13, 2024
## Linked issues

- Resolve #3188
- Resolve #3183
- #3594
- Ajout de documentation dans le Makefile:
![Screenshot from 2024-09-13
11-27-53](https://github.com/user-attachments/assets/ec585dca-0432-4c6e-b468-b74623ff3bba)

----

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

@louptheron c'est bon, je vois juste un souci de décalage dans le résumé de l'historique quand il y a plusieurs types de signalement, cf. cette capture :
image

@louptheron
Copy link
Collaborator

Ok en inté

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment