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.

Sign up for GitHub

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

Jump to bottom

Missions - Afficher l'enregistrement automatique seulement sur les missions en cours #2770

Closed
louptheron opened this issue Dec 18, 2023 · 0 comments · Fixed by #2720
Closed

Missions - Afficher l'enregistrement automatique seulement sur les missions en cours #2770

louptheron opened this issue Dec 18, 2023 · 0 comments · Fixed by #2720
Assignees
@AdelineCelier
Labels
dev feat. enhancement feature enhancement

Comments

@louptheron
Copy link
Collaborator

On affiche le bouton "Enregistrer" quand

  • La mission est terminée (c-a-d que la today > date de fin de mission) et
  • La mission est "clôturée"
@louptheron louptheron added feat. enhancement feature enhancement dev labels Dec 18, 2023
@louptheron louptheron mentioned this issue Dec 18, 2023
Merged
1 task
louptheron added a commit that referenced this issue Dec 22, 2023
@louptheron
[Missions] Rafraichissement & Enregistrement automatique (UPDATE ENV …
078dcc8 
…VAR) (#2720)

## Linked issues

- Resolve #2697
- Ajout du feature flag `FRONTEND_MISSION_AUTO_SAVE_ENABLED`
- Resolve #2770
- Resolve #2698 

Quelques notes :
- `isDirty` correspond maintenant au fait que les formulaires soient en
erreur (et du coup non-enregistrés en backend): la modale de fermeture
s'affiche donc quand le formulaire est en erreur
- Le `EVENT_SOURCE.addEventListener(MISSION_UPDATE_EVENT, listener)` est
maintenant dans le formulaire Formik
(`MissionForm/MainForm/FormikSyncMissionFields.tsx`) et mets à jour
directement les champs avec `setFieldValue`. Cela permet de ne pas
perdre le focus (re-render le formulaire) quand on reçoit une MAJ d'un
autre poste.
- Les tests E2E utilisent maintenant `waitForLastRequest` car il y a une
requête qui part à chaque modification, un `cy.wait('@updatedMission')`
ne pouvait pas attendre la bonne requête
- On garde le bouton `Clôturer` en attendant de changer les statuts de
missions
- J'ai essayé en vain de supprimer le state redux `draft` (la mission)
mais je dois le garder pour les actions transverses (i.e formulaire
d'action qui met à jour le formulaire de mission, etc.)
- à chaque `onChange` des formulaires de mission ou d'action, le state
de `MissionForm` et le state redux `draft` sont mis à jour, même lorsque
les formulaires sont en erreurs. **La requête d'`update` ne part que
quand la validation sur les formulaires passe**
- Lors de l'édition d'une mission, tout ne s'actualise pas en temps réél
sur la carte : il faut attendre que le formulaire soit valide et que le
polling des missions ait lieu pour que le `Overlay` sur une mission soit
à jour.

----

- [x] Tests E2E (Cypress)
@louptheron louptheron moved this to Done in Board MonitorFish May 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AdelineCelier AdelineCelier

Labels
dev feat. enhancement feature enhancement
Projects
Board MonitorFish
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Missions] Rafraichissement & Enregistrement automatique (UPDATE ENV VAR) MTES-MCT/monitorfish
2 participants
@louptheron @AdelineCelier