[MISSIONS/ACTION] Garder le formulaire des actions ouverts même si on change de fenêtre #1461
Conversation
maximeperraultdev
changed the title
| @@ -59,33 +58,26 @@ export function ActionForm({ currentActionIndex, setCurrentActionIndex }: Action | |||
| return ( | |||
| <FormWrapper> | |||
| <ControlForm | |||
| key={actionIdField.value} | |||
There was a problem hiding this comment.
il faut laisser les key sinon ça pose des problèmes quand tu as plusieurs actions du même type
| @@ -55,13 +55,13 @@ import { | |||
| import { ActionTheme } from '../Themes/ActionTheme' | |||
|
|
|||
| export function ControlForm({ | |||
| currentActionIndex, | |||
| currentActionId, | |||
There was a problem hiding this comment.
Alors je suis d'accord que c'est l'id, mais c'est aussi l'index du tableau des différentes actions, je trouve ça plus clair de laisser la notion d'index, parce que je trouve que setCurrentActionId ne reflète pas la réalite. On en modifie pas l'id de l'action "courante" mais bien l'index dans la liste des actions.
maximeperraultdev
changed the title
maximeperraultdev
force-pushed
the
maxime/feat/1187/garder_formulaire_ouvert
branch
3 times, most recently
from
0844ec8
to
34ccd3d
Compare
| Valider l'infraction | ||
| </Button> | ||
| </ButtonContainer> | ||
| </FormWrapper> | ||
| ) | ||
|
|
||
| function isInfractionFormInvalid(errorsForm: FormikErrors<Mission>) { |
There was a problem hiding this comment.
on a pas l'habitude de mettre les fonctions après le return, j'ai peur que ça devienne un peu le bazar si on fait pas tous pareil. on va se retrouver avec des fichiers qui n'ont pas la même structure et ça risque de devenir vite un enfer
| @@ -27,7 +30,9 @@ export function useSyncFormValuesWithRedux(isAutoSaveEnabled: boolean) { | |||
|
|
|||
| const isFormDirty = isMissionFormDirty() | |||
|
|
|||
| dispatch(missionFormsActions.setMission({ engagedControlUnit, isFormDirty, missionForm: newValues })) | |||
| dispatch( | |||
There was a problem hiding this comment.
je trouve ca étrange d'aller récupérer la donnée pour la réinjecter telle qu'elle. On ne peut pas simplement ne pas passer la donnée?
…ype and information entered
…ype and information entered
…ype and information entered
maximeperraultdev
force-pushed
the
maxime/feat/1187/garder_formulaire_ouvert
branch
from
80d96c6
to
8629dae
Compare
Related Pull Requests & Issues