Add possibility for setting required fields in ContactInfoSection

Since it should be possible to vary the required fields depending on the context. In this case user creation vs user info form.
danskernesdigitalebibliotek · Oct 18, 2023 · f843c31 · f843c31
1 parent 5ef6b0b
commit f843c31
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/apps/create-patron-user-info/UserInfo.tsx b/src/apps/create-patron-user-info/UserInfo.tsx
@@ -75,6 +75,7 @@ const UserInfo: FC<UserInfoProps> = ({ cpr }) => {
             inLine
             changePatron={changePatron}
             patron={patron}
+            requiredFields={["email"]}
           />
           <PincodeSection required changePincode={setPin} />
           {t("createPatronChangePickupHeaderText") && (

diff --git a/src/components/contact-info-section/ContactInfoSection.tsx b/src/components/contact-info-section/ContactInfoSection.tsx
@@ -13,13 +13,15 @@ interface ContactInfoSectionProps {
   inLine: boolean;
   changePatron: ChangePatronProps;
   showCheckboxes: boolean;
+  requiredFields?: ("email" | "phone")[];
 }
 
 const ContactInfoSection: FC<ContactInfoSectionProps> = ({
   patron,
   inLine,
   changePatron,
-  showCheckboxes
+  showCheckboxes,
+  requiredFields = []
 }) => {
   const t = useText();
   const inputsClass = clsx("dpl-input", { input__desktop: inLine });
@@ -42,6 +44,7 @@ const ContactInfoSection: FC<ContactInfoSectionProps> = ({
           className={inputsClass}
           changePatron={changePatron}
           patron={patron}
+          isRequired={requiredFields.includes("phone")}
           showCheckboxes={showCheckboxes && textNotificationsEnabledConfig}
         />
         <ContactInfoEmail
@@ -50,6 +53,7 @@ const ContactInfoSection: FC<ContactInfoSectionProps> = ({
           })}
           changePatron={changePatron}
           patron={patron}
+          isRequired={requiredFields.includes("email")}
           showCheckboxes={showCheckboxes}
         />
       </ContactInfoInputs>