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] Enregistrement automatique #1109

Merged

Conversation

louptheron
Copy link
Collaborator

@louptheron louptheron commented Jan 4, 2024

Related Pull Requests & Issues

  • Resolve Missions - Enregistrement automatique #985
  • Ajouter le feature flag REACT_APP_MISSION_AUTO_SAVE_ENABLED
  • Refactoring de la mise-à-jour de Formik avec un setValue au lieu de initialValues (qui lui reste freeze)

Quelques notes :

  • Le schéma de validation NewMissionSchema devra être réduit pour ne pas bloquer l'enregistrement auto. à la création d'une mission : pour l'instant endDateTimeUtc et openBy sont toujours required - à enlever lors de la 1ère MEP d'utilisation de la fonctionnalité.

Ajout d'un test E2E multi-windows avec Puppeteer :

Screencast.from.16-01-2024.09.36.38.webm

  • Tests E2E (Cypress)

@louptheron louptheron marked this pull request as ready for review January 10, 2024 09:52
@louptheron louptheron force-pushed the loup/add-mission-form-auto-save-and-update-refactoring branch from 388dadb to 35b45d8 Compare January 10, 2024 09:54
@louptheron louptheron force-pushed the loup/add-mission-form-auto-save-and-update-refactoring branch 2 times, most recently from b6f92d0 to 620a6cd Compare January 11, 2024 16:52
@claire2212
Copy link
Collaborator

claire2212 commented Jan 12, 2024

@louptheron :

  • problème de synchronisation sur certains champs (clôture, signalements rattachés/ détachés, ?) avec une erreur 404 quand la mission est ouverte sur 2 postes.
    Le 2 ème poste fait un PUT avec les nouvelles reçues

@louptheron
Copy link
Collaborator Author

louptheron commented Jan 12, 2024

@louptheron :

* problème de synchronisation sur certains champs (clôture, signalements rattachés/ détachés, ?) avec une erreur 404 quand la mission est ouverte sur 2 postes.
  Le 2 ème poste fait un PUT avec les nouvelles reçues
  • Le souci de la côture semble fixé, @claire2212 tu peux re-tester ?
  • Pour les signalements, le formulaire ouvert ne se met pas à jour de façon automatique, il faut ré-ouvrir la mission (commes les actions) : je propose de faire commes les actions et de faire ça plus tard, étant donné qu'un superviseur travaille habituellement sur une mission en solo ?

@louptheron louptheron force-pushed the loup/add-mission-form-auto-save-and-update-refactoring branch from 620a6cd to f965364 Compare January 12, 2024 15:42
@louptheron louptheron force-pushed the loup/add-mission-form-auto-save-and-update-refactoring branch from 1527cff to 07f95aa Compare January 17, 2024 14:29
@louptheron louptheron merged commit 19ca4b9 into main Jan 25, 2024
10 checks passed
@louptheron louptheron deleted the loup/add-mission-form-auto-save-and-update-refactoring branch January 25, 2024 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missions - Enregistrement automatique
2 participants