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

[Missions] Rafraichissement & Enregistrement automatique (UPDATE ENV VAR) #2720

Merged
merged 34 commits into from
Dec 22, 2023

Conversation

louptheron
Copy link
Collaborator

@louptheron louptheron commented Nov 23, 2023

Linked issues

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.

  • Tests E2E (Cypress)

@louptheron louptheron force-pushed the loup/auto-saving branch 2 times, most recently from 09cd83f to a593036 Compare December 6, 2023 16:56
@louptheron louptheron changed the title [Missions] Enregistrement automatique [Missions] Rafraichissement & Enregistrement automatique Dec 13, 2023
@louptheron louptheron marked this pull request as ready for review December 13, 2023 16:10
@louptheron louptheron changed the title [Missions] Rafraichissement & Enregistrement automatique [Missions] Rafraichissement & Enregistrement automatique (BREAKING) Dec 20, 2023
@louptheron louptheron changed the title [Missions] Rafraichissement & Enregistrement automatique (BREAKING) [Missions] Rafraichissement & Enregistrement automatique (BREAKING CHANGES) Dec 20, 2023
@louptheron louptheron changed the title [Missions] Rafraichissement & Enregistrement automatique (BREAKING CHANGES) [Missions] Rafraichissement & Enregistrement automatique (UPDATE ENV VAR) Dec 20, 2023
Copy link
Collaborator

@thoomasbro thoomasbro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool!

Copy link

sonarcloud bot commented Dec 22, 2023

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

6 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@louptheron louptheron merged commit 078dcc8 into master Dec 22, 2023
23 of 24 checks passed
@louptheron louptheron deleted the loup/auto-saving branch December 22, 2023 10:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants