add missing values in hook deps

gip-inclusion · Dec 10, 2024 · 362a796 · 362a796
1 parent 1314375
commit 362a796
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 9 deletions.
diff --git a/front/src/app/components/forms/convention/ConventionForm.tsx b/front/src/app/components/forms/convention/ConventionForm.tsx
@@ -347,7 +347,7 @@ export const ConventionForm = ({
       "partialConventionInUrl",
     );
     dispatch(conventionSlice.actions.setCurrentStep(1));
-  }, []);
+  }, [dispatch]);
 
   useEffect(() => {
     if (mode !== "create") {
@@ -361,7 +361,7 @@ export const ConventionForm = ({
     if (fetchedConvention) {
       reset(fetchedConvention);
     }
-  }, [fetchedConvention, methods.reset]);
+  }, [fetchedConvention, reset]);
 
   return (
     <FormProvider {...methods}>

diff --git a/front/src/app/components/forms/convention/sections/beneficiary/BeneficiaryFormSection.tsx b/front/src/app/components/forms/convention/sections/beneficiary/BeneficiaryFormSection.tsx
@@ -76,18 +76,19 @@ export const BeneficiaryFormSection = ({
     if (hasBeneficiaryRepresentativeData(initialBeneficiaryRepresentative)) {
       dispatch(conventionSlice.actions.isMinorChanged(true));
     }
-  }, [initialBeneficiaryRepresentative]);
+  }, [dispatch, initialBeneficiaryRepresentative]);
 
   useEffect(() => {
     if (userFieldsAreFilled) {
+      const { firstName, lastName, email } = connectedUser;
       const valuesToUpdate = {
-        "signatories.beneficiary.firstName": connectedUser.firstName,
-        "signatories.beneficiary.lastName": connectedUser.lastName,
-        "signatories.beneficiary.email": connectedUser.email,
+        "signatories.beneficiary.firstName": firstName,
+        "signatories.beneficiary.lastName": lastName,
+        "signatories.beneficiary.email": email,
       };
       keys(valuesToUpdate).forEach((key) => setValue(key, valuesToUpdate[key]));
     }
-  }, [userFieldsAreFilled]);
+  }, [userFieldsAreFilled, connectedUser, setValue]);
 
   useEffect(() => {
     const initialValues = values.signatories.beneficiaryCurrentEmployer;
@@ -100,7 +101,11 @@ export const BeneficiaryFormSection = ({
           }
         : undefined,
     );
-  }, [hasCurrentEmployer]);
+  }, [
+    hasCurrentEmployer,
+    setValue,
+    values.signatories.beneficiaryCurrentEmployer,
+  ]);
 
   const levelsOfEducationToSelectOption = levelsOfEducation.map(
     (level: string) => ({ label: level, value: level }),
@@ -122,7 +127,7 @@ export const BeneficiaryFormSection = ({
       setIsMinorAccordingToAge(newIsMinor);
       dispatch(conventionSlice.actions.isMinorChanged(newIsMinor));
     },
-    [dispatch, values.dateStart],
+    [dispatch, values.dateStart, setValue, getValues],
   );
 
   useEffect(() => {