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 – Pré-remplir le contrôle avec les coordonnées du signalement rattaché ? #1146

Closed
AdelineCelier opened this issue Jan 24, 2024 · 7 comments · Fixed by #1198 or #1895
Assignees
Labels
dev feat. enhancement Amélioration/évolution d'une fonctionnalité

Comments

@AdelineCelier
Copy link
Collaborator

AdelineCelier commented Jan 24, 2024

Reprise du commentaire de @manonolivier dans le ticket #1138.

On a retesté avec @claire2212 , quand on crée un contrôle à partir d'un signalement dans une mission, les champs suivants sont pré-remplis (s'ils ont bien été saisis dans le signalement bien sûr) :

  • thématique
  • sous-thématique
  • type de cible
    Je me rends compte qu'on n'avait pas mentionné ce point-là dans la présentation qu'on vous a faite de la fonctionnalité fin 2023. J'ai mis à jour le document, n'hésitez surtout pas à aller le re-consulter quand vous avez un doute, il est fait pour ça ;)

On avait délibérément évité le remplissage auto du point de contrôle à partir du point de signalement en se disant que la cible peut bouger et donc ne pas être au même endroit au moment du contrôle.
Et de plus il y a en effet le cas de la zone : on pourrait dire qu'on prend par défaut le centre de la zone pour le point de contrôle, mais ça ne me semble pas vraiment gage de précision.

Perso j'ai l'impression qu'il y a un petit risque à automatiser ce remplissage du point de contrôle, mais dites-moi si vous n'êtes pas d'accord.
Et est-ce que vous pouvez retester de votre côté le pré-remplissage des autres infos et nous dire si ça fonctionne bien ?

@AdelineCelier AdelineCelier changed the title Signalements – Lorsqu'on crée un contrôle à partir d'un signalement rattaché à une mission, pré-remplir le contrôle avec les infos du signalement ? Signalements – Lorsqu'on crée un contrôle à partir d'un signalement dans une mission, pré-remplir le contrôle avec les infos du signalement ? Jan 24, 2024
@AdelineCelier AdelineCelier changed the title Signalements – Lorsqu'on crée un contrôle à partir d'un signalement dans une mission, pré-remplir le contrôle avec les infos du signalement ? Signalements – Pré-remplir le contrôle avec les coordonnées du signalement rattaché ? Jan 24, 2024
@manonolivier
Copy link
Collaborator

@AdelineCelier @claire2212

Les thématiques et type de cible se remplissent bien automatiquement.

La grande majorité des signalements seront contrôlés au même endroit mais j'en parle lors de notre point hebdo pour voir ce que les autres en pensent et je reviens vers vous.

Je pensais à un truc sur le pré remplissage (désolée s'il fallait plutôt le dire dans un autre ticket vu que ce n'est pas lié à la position) : est-ce que c'est possible de pouvoir également remplir automatiquement l'identité de la cible ?
Je pense surtout aux moments où ça va être le rush cet été avec les mouillages med. Il faut qu'on remplisse : NOM - IMMATRICULATION - TAILLE ça va être chronophage de devoir tout remplir de nouveau dans le contrôle alors qu'on l'a déjà rempli dans le signalement.
Le fait de devoir remplir à nouveau des informations est également une source d'erreur, je pense qu'il y aura quelques incohérences entre les infos du contrôle et les infos du signalement.
La fiche "détails de la cible en infraction" dans la mission se ferme à chaque fois qu'on va sur l'action de signalement, ça rajoute des clics en plus à faire.
==> Si c'est trop complexe à mettre en place, une solution serait d'afficher la fiche signalement sur le SIG et faire un copier coller manuel directement sur la fiche mission. Mais si c'était possible d'automatiser, on gagnerait un temps et une qualité précieuse.

@claire2212
Copy link
Collaborator

claire2212 commented Feb 5, 2024

Salut @manonolivier , on peut tout à fait pré-remplir une infraction avec les infos : NOM - IMMATRICULATION - TAILLE. J'ai juste une question sur la taille :

  • Côté signalement le champ "taille" est un champ de type nombre, c'est à dire que vous renseignez à la main la taille (ex: 12, 24, ...)
  • Côté mission le champ "taille" est un sélecteur dans lequel vous choisissez une fourchette de taille (de 12 à 24m, moins de 12m, ...)

Est-ce que vous souhaitez :

  • qu'on garde ça comme ça et suivant la taille saisie dans le signalement je calcule la taille correspondant côté formulaire d'infraction (ex: 17mètres renseignés dans le signalement = de12 à 24m dans le formulaire d'infraction)
  • Qu'on harmonise en gardant la solution du sélecteur de fourchette (de 12 à 24m, moins de 12m, ...)
  • Qu'on harmonise en gardant la solution du champ de type nombre pou rune saisie manuelle (ex: 12, 24, ...)

@claire2212 claire2212 self-assigned this Feb 5, 2024
@claire2212 claire2212 added feat. enhancement Amélioration/évolution d'une fonctionnalité and removed question labels Feb 5, 2024
@claire2212 claire2212 added dev and removed question labels Feb 13, 2024
claire2212 added a commit that referenced this issue Mar 6, 2024
…1198)

## Related Pull Requests & Issues

- Resolve #1146 

----

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

Comme discuté ensemble à l'instant @claire2212, ce ticket est bon on peut le passer.
En revanche on s'est rendu compte que le fait que les champs des infos de cibles ne soit pas iso entre les signalements et les actions dans les missions peut prêter à confusion et qu'il vaudrait mieux les aligner.
J'ouvre un nouveau ticket sur le sujet pour demander son avis au CACEM.

@manonolivier manonolivier moved this to Done in Retours CACEM Jul 26, 2024
@AdelineCelier
Copy link
Collaborator Author

@manonolivier on a eu à nouveau un retour sur le fait de pré-remplir automatiquement le point de contrôle avec le point de signalement. Cela soulevait plusieurs questions (cf. mon 1er commentaire tout en haut du ticket). Tu devais en reparler avec l'équipe CACEM pour trancher, est-ce que vous l'avez fait ? Quelle conclusion au final ?

@manonolivier
Copy link
Collaborator

@AdelineCelier Revu en point hebdo le 23/10 avec Pierre et Alice. Nous préférons que ce soit rentré automatiquement car le lieu du signalement est quasi tout le temps le lieu du contrôle et ça permet d'éviter les erreurs.
Pour éviter d'oublier de modifier le point de contrôle quand il le faut, est-il possible de mettre une sorte de warning "position rentrée automatiquement" qui servirait de rappel au cas où il faut modifier le point

@AdelineCelier
Copy link
Collaborator Author

Hello @manonolivier, avec un peu de retard : ok je remets ce ticket dans "à développer".
Je vais regarder pour l'ajout d'un petit encart. Il pourrait concerner toutes les infos pré-remplies automatiquement dans l'infraction à partir du signalement, pas seulement la position.

@AdelineCelier AdelineCelier reopened this Nov 21, 2024
@AdelineCelier AdelineCelier moved this from Terminé to Pris en compte – À développer in Retours CACEM Nov 21, 2024
@AdelineCelier AdelineCelier moved this from Done to Backlog in Kanban MonitorEnv Nov 21, 2024
@AdelineCelier
Copy link
Collaborator Author

@claire2212 bon au final je pense que le plus simple est de mettre un encart bleu sous le point de contrôle quand celui-ci a automatiquement été calculé à partir de la position du signalement. L'encart disparaît si l'utilisateur supprime ou change la position du point de contrôle.

Écran ici.

@claire2212 claire2212 moved this from Backlog to In progress in Kanban MonitorEnv Nov 28, 2024
@claire2212 claire2212 moved this from In progress to In Code Review in Kanban MonitorEnv Nov 29, 2024
@claire2212 claire2212 moved this from In Code Review to To be deployed in INT for review in Kanban MonitorEnv Dec 3, 2024
@claire2212 claire2212 moved this from Pris en compte – À développer to Développement en cours in Retours CACEM Dec 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev feat. enhancement Amélioration/évolution d'une fonctionnalité
Projects
Status: To be deployed in INT for review
Status: Développement en cours
3 participants