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

Mission - L'ouverture d'une mission alors que le formulaire est déjà ouvert écrase les actions #2933

Closed
1 task
louptheron opened this issue Feb 14, 2024 · 0 comments · Fixed by #2815
Closed
1 task
Labels
bug Something isn't working dev

Comments

@louptheron
Copy link
Collaborator

louptheron commented Feb 14, 2024

Un bug en production depuis le 07/02/2024 écrase les contrôles quand le formulaire d'une mission est déjà ouvert et qu'on édite une mission depuis la vue navire.

Scénario

  1. J'édite une mission dans le formulaire (que je ne modifie pas)
  2. Dans la liste des contrôles d'un navire, je clique sur "Ouvrir le contrôle"
  3. Je clique sur "Enregistrer"
  4. Tous les contrôles de l'anciennes missions sont ajoutés dans la mission qui vient d'être ouverte depuis la fiche navire

Problème

Si le composant MissionForm est déjà monté, il ne se met pas à jour. (c'est pourquoi ce bug n'apparait pas depuis la vue liste) :

  // ---------------------------------------------------------------------------
  // DATA INITIALIZATION ON COMPONENT MOUNT

  useEffect(() => {
    // We hide selected missions geometries and overlays on map
    dispatch(missionActions.unsetSelectedMissionGeoJSON())

    if (isLoading) {
      return
    }

    if (mainFormValues) {    <----- /!\  ICI
      return
    }

Le commit a été ajouté le 13/12/2023 (df3f116fda3943de44a9b959bbbc6c21dfc3d765), donc il est en production depuis le 07/02/2024.

Résolution

Le passage en thunk par le PR #2815 résout ces soucis ?

To do

  • Ajouter un test E2E avec Pupeteer
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working dev
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants