-
Notifications
You must be signed in to change notification settings - Fork 1
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
Points d'intérêt – Amélioration de la fonctionnalité #1214
Comments
…es points sur la carte, changer l'icone (#1397) ## Related Pull Requests & Issues - Resolve #1214 ---- - [ x] Tests E2E (Cypress) ## What have I done in this PR - Refacto EditInterestPoint to use the same component than Mission's - Rewriting test to make e2e cypress tests idempotent - Upgrade cypress to 13.10.0 because of a bug when several windows are open - A little bit of BSR *(Boy Scout Rules)* : typo, extract methods for lisibility, small css fixes
@maximeperraultdev les points cochés du ticket sont bons, en revanche je viens juste de remarquer deux trucs (qui je pense ne sont pas bloquants pour la mep) :
|
@AdelineCelier ça marche je repasse dessus |
Suite à point avec Maxime :
|
## Related Pull Requests & Issues - Resolve #1214 - See last comments ---- - [x] Tests E2E (Cypress) ## What have I done in this PR Improve Interest point features fixing async bugs Moved file and directory to stick to the clean architecture
@maximeperraultdev reste les points dont on vient de parler ensemble :
|
## Related Pull Requests & Issues - Resolve #1214 ---- - [x] Tests E2E (Cypress) ---- ## What have I done in this PR - [x] FIX: quand on édite un point déjà créé puis qu'on le supprime, le nouveau point qu'on créé ensuite ne s'enregistre pas (juste un point sans étiquette, qui disparaît quand on recharge la page) - [x] FEAT: possible de rajouter un bouton secondaire "supprimer le point" en-dessous de "enregistrer le point" ? (mêmes dimensions que les autres boutons, même écartement qu'entre les deux boutons actuels) - [x] FEAT: mise en place de l'action "Annuler" quand on édite un point d'intérêt - [x] REFACTO: Création de use case (action redux) à la place des useEffect - [x] FIX: Trouver une action globale pour ne pas avoir à nettoyer les points rémanents de la carte "à la main"
C'est good ! |
Après discussion avec le CACEM, il se trouve que la fonctionnalité est très utilisée comme post-it/pense-bête à tout faire.
Exemples d'usages donnés :
Plusieurs points d'amélioration possibles de la fonctionnalité :
Quick wins
Retirer les types de points, qui ne sont pas utilisés par le CACEM 👍🏻
Changer la couleur des points sur la carte, pour qu'ils soient plus lisibles (cf. ticket 751 du CACEM) - cf cet écran
!(https://github.com/MTES-MCT/monitorenv/assets/71878648/eab63095-b683-409b-ab05-9e9f1e6c4feb)
Pouvoir masquer/afficher tous les points d'intérêt (cf. ticket 745 du CACEM) - cf ce même écran 👍🏻
Dans un second temps si complexe
The text was updated successfully, but these errors were encountered: