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] [FIX] Attacher les reportings à l'ouverture du formulaire de mission plutot qu'au changement de fenetre #1503

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

maximeperraultdev
Copy link
Collaborator

@maximeperraultdev maximeperraultdev commented Jun 25, 2024

Related Pull Requests & Issues


  • 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

@maximeperraultdev maximeperraultdev changed the title fix: Synchronize field update with save [MISSION] [FIX] Modification des champs puis sauvegarde Jun 25, 2024
@maximeperraultdev maximeperraultdev force-pushed the maxime/fix/synchronize_missions_fields branch 2 times, most recently from 1f441b5 to bd55a8d Compare July 11, 2024 08:17
@maximeperraultdev maximeperraultdev force-pushed the maxime/fix/synchronize_missions_fields branch 2 times, most recently from 6aa9237 to d9e38eb Compare September 19, 2024 07:48
@maximeperraultdev maximeperraultdev changed the title [MISSION] [FIX] Modification des champs puis sauvegarde [MISSION] [FIX] Attacher les reportings à l'ouverture du formulaire de mission plutot qu'au changement de fenetre Sep 19, 2024
@claire2212
Copy link
Collaborator

@maximeperraultdev je viens de tester et ça met une erreur en console
Capture d’écran 2024-09-19 à 13 28 58

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
Copy link
Collaborator Author

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

@claire2212
Copy link
Collaborator

@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
Copy link
Collaborator Author

@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.

@maximeperraultdev maximeperraultdev force-pushed the maxime/fix/synchronize_missions_fields branch 2 times, most recently from 5475836 to ad669c1 Compare October 23, 2024 08:03
@maximeperraultdev maximeperraultdev force-pushed the maxime/fix/synchronize_missions_fields branch from ad669c1 to 038ea42 Compare October 28, 2024 08:17
Copy link

sonarcloud bot commented Oct 28, 2024

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.

[Mission] - Garder ouverts les formulaires d'actions (et infractions) pendant la navigation
3 participants