[MISSION] [FIX] Attacher les reportings à l'ouverture du formulaire de mission plutot qu'au changement de fenetre

[maximeperraultdev]

Related Pull Requests & Issues

• Resolve [Mission] - Garder ouverts les formulaires d'actions (et infractions) pendant la navigation #1187 second commentaire de Adeline

---

• Tests E2E (Cypress)

---

What have I done

• Déplacement de la mise à jour des signalements rattachés de la navigation entre onglets vers le formulaire des missions

[claire2212]

@maximeperraultdev je viens de tester et ça met une erreur en console

Ce qui est bizarre c'est le dispatch comme ça qui n'est ni dans une fonction ni dans un useEffect, est qui est donc appelé à chaque re-render de fichier

Et sinon je me suis rendue compte que le bouton pour fermer la modal de rattachement d'un signalement ne fonctionne pas. Il manque la fonction dans le fichier AttachReportingToMissionModal.tsx. Dans le composant MapIntercation il faudrait ajouter onCncel={cancelReportingToAttach}

et la fonction :

  const cancelReportingToAttach = () => {
    dispatch(updateMapInteractionListeners(MapInteractionListenerEnum.NONE))
    dispatch(attachReportingToMissionSliceActions.setAttachedReportings(initialAttachedReportings))
  }

[maximeperraultdev]

@claire2212 J'ai ajouté le fix pour annuler le rattachement des signalements et j'ai wrappé le dispatch dans un useEffect !

[claire2212]

@maximeperrault je viens de retester et ca ne fonctionne pas correctement.
J'ai créée une mission, j'ai voulu ajouter trois signalements en une seule fois, quand je retourne dans la mission, la timeline fait des trucs bizarres elle affiche/masque certains signalements sans fin

Enregistrement.de.l.ecran.2024-09-20.a.16.54.30.mov

[maximeperraultdev]

@claire2212 j'avais repéré un bug quand tu detache le signalement de la mission mais pas a ce point là. Je regardes apres les tableaux de bord.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud