fix: bug prod etape 5 et 6 avec bouton oui / non (#41)

Author: pom421

src/app/simulation/etapes/Step5/index.tsx

@@ -35,11 +35,12 @@ const schemaNon = z.object({
       return { message: context.defaultError };
     },
   }),
+  espacesExterieursCommuns: z.undefined(),
 });
 
 export const Step5 = () => {
   const [radioState, setRadioState] = useState<(typeof OuiNonLabels)[number] | undefined>();
-  const { store } = usePacoupaSessionStorage();
+  const { store, setStore } = usePacoupaSessionStorage();
 
   useEffect(() => {
     setRadioState((store.possedeEspacesExterieursCommuns as (typeof OuiNonLabels)[number]) ?? "Oui");
@@ -52,8 +53,11 @@ export const Step5 = () => {
           el.checked = false;
         },
       );
+      setStore({ ...store, espacesExterieursCommuns: undefined });
     }
-  }, [radioState]);
+    // Doesn't depend on store and make a recursive call if present.
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, [radioState, setStore]);
 
   return (
     <>

src/app/simulation/etapes/Step6/index.tsx

@@ -35,25 +35,28 @@ const schemaNon = z.object({
       return { message: context.defaultError };
     },
   }),
+  espacesExterieursPersonnels: z.undefined(),
 });
 
 export const Step6 = () => {
   const [radioState, setRadioState] = useState<(typeof OuiNonLabels)[number] | undefined>();
-  const { store } = usePacoupaSessionStorage();
+  const { store, setStore } = usePacoupaSessionStorage();
 
   useEffect(() => {
-    setRadioState((store.possedeEspacesExterieursPersonnels as (typeof OuiNonLabels)[number]) ?? "Oui");
+    setRadioState((store.possedeEspacesExterieursCommuns as (typeof OuiNonLabels)[number]) ?? "Oui");
   }, [store]);
-
   useEffect(() => {
     if (radioState === "Non") {
       (document.getElementsByName("espacesExterieursPersonnels") as NodeListOf<HTMLInputElement>).forEach(
         (el: HTMLInputElement) => {
           el.checked = false;
         },
       );
+      setStore({ ...store, espacesExterieursPersonnels: undefined });
     }
-  }, [radioState]);
+    // Doesn't depend on store and make a recursive call if present.
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, [radioState, setStore]);
 
   return (
     <>