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

Impossible de modifier une mission avec MED en attente #1363

Closed
manonolivier opened this issue May 8, 2024 · 5 comments · Fixed by #1372 or #1377
Closed

Impossible de modifier une mission avec MED en attente #1363

manonolivier opened this issue May 8, 2024 · 5 comments · Fixed by #1372 or #1377
Assignees
Labels
feat. enhancement Amélioration/évolution d'une fonctionnalité Synchro Mission

Comments

@manonolivier
Copy link
Collaborator

manonolivier commented May 8, 2024

Je faisais le point sur les missions qui sont "à compléter" par le CACEM (il manque des trigrammes) et je suis arrivée à une mission où il manquait des trigrammes, je les ais ajouté mais je ne peux pas enregistrer les modifications que j'ai fait parce que MonitorEnv ne veut pas laisser la mise en demeure en attente.
image

@manonolivier manonolivier created this issue from a note in Retours CACEM (Remontée MonitorEnv) May 8, 2024
@claire2212
Copy link
Collaborator

Salut @manonolivier est-ce que tu peux me donner plus de détails sur la mission (unité et dates notamment) en question que je regarde d'où peut venir le problème ?
Au moment de ta saisie la mission était en cours ou terminée? Parce que si elle était terminée c'est normal que tu n'ai pas pu mettre En attente car on a gardé le fonctionnement de la mission clôturée (donc mission terminée = obligation de mettre Oui ou Non pour le champ mise en demeure)

@manonolivier
Copy link
Collaborator Author

Salut @claire2212, ça concerne toutes les missions avec un PV ou une MED en attente. Pour exemple : BN St Martin le 15/04 de 15h00 à 17h00

La mission est en effet terminée.
Le "en attente" peut durer longtemps (ils peuvent avoir besoin de faire une enquête poussée au bureau et donc la mission terrain est terminée). Sauf qu'entre temps, je ne peux rien modifier (ajouter un trigramme, modifier le nombre de contrôle, la thématique, etc).
Pour être précise, je peux modifier mais ça ne veut pas enregistrer les modifications.
Alors que je ne veux pas forcément que ça parte dans les stats sur metabase directement (anciennement version clôturée) mais je veux préciser des infos a posteriori et laisser en attente quand même.

D'ailleurs, un "en attente" sur l'action 1 m'empêche de modifier des choses dans l'action 2 (action 2 qui n'a pas de en attente)

Pas facile de l'expliquer par écrit, si ce n'est pas clair, on peut s'appeler

@AdelineCelier
Copy link
Collaborator

AdelineCelier commented May 13, 2024

@manonolivier on vient de refaire le point avec @claire2212 , on va modifier le comportement de l'enregistrement quand la mission est terminée : il sera possible d'enregistrer la mission (que ce soit enregistrement auto u en enregistrement manuel) quand bien même tous les champs en rouge ne seront pas remplis (ex. le champ "en attente", qui restera en rouge puisqu'on attend la réponse). L'état des données de la mission restera "à compléter".

@manonolivier
Copy link
Collaborator Author

Salut @AdelineCelier et @claire2212
C'est top, merci !

@claire2212 claire2212 self-assigned this May 14, 2024
@claire2212 claire2212 added this to Todo in Kanban May 14, 2024
@claire2212 claire2212 added the feat. enhancement Amélioration/évolution d'une fonctionnalité label May 14, 2024
@claire2212 claire2212 moved this from Todo to In progress in Kanban May 14, 2024
claire2212 added a commit that referenced this issue May 14, 2024
…quants (#1372)

## Related Pull Requests & Issues

- Resolve #1363 

----

- [ ] Tests E2E (Cypress)
@claire2212 claire2212 moved this from In progress to In Code Review in Kanban May 14, 2024
@claire2212 claire2212 moved this from In Code Review to To be deployed in INT for review in Kanban May 14, 2024
claire2212 added a commit that referenced this issue May 16, 2024
… sont manquants (#1377)

## Related Pull Requests & Issues

- Resolve #1363

----

- [ ] Tests E2E (Cypress)
@claire2212 claire2212 moved this from To be deployed in INT for review to To Review in INT in Kanban May 16, 2024
@claire2212 claire2212 moved this from Remontée MonitorEnv to Développement en cours in Retours CACEM May 16, 2024
@AdelineCelier
Copy link
Collaborator

Claire c'est bon, ça fonctionne bien, en enregistrement auto comme en enregistrement manuel !

@AdelineCelier AdelineCelier moved this from To Review in INT to To deploy in PROD in Kanban May 16, 2024
@claire2212 claire2212 moved this from Développement en cours to Done in Retours CACEM May 17, 2024
@claire2212 claire2212 moved this from To deploy in PROD to Done in Kanban May 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment