diff --git a/frontend/app/routes/protected/person-case/personal-info.tsx b/frontend/app/routes/protected/person-case/personal-info.tsx index 8f14356e..a7602b22 100644 --- a/frontend/app/routes/protected/person-case/personal-info.tsx +++ b/frontend/app/routes/protected/person-case/personal-info.tsx @@ -86,7 +86,7 @@ export async function action({ context, request }: Route.ActionArgs) { case 'next': { const schema = v.object({ - firstNamePreviouslyUsed: v.optional(v.array(v.string())), + firstNamePreviouslyUsed: v.optional(v.array(v.pipe(v.string(), v.trim()))), lastNameAtBirth: v.pipe( v.string(t('protected:personal-information.last-name-at-birth.required')), v.trim(), @@ -97,7 +97,7 @@ export async function action({ context, request }: Route.ActionArgs) { ), v.regex(REGEX_PATTERNS.NON_DIGIT, t('protected:personal-information.last-name-at-birth.format')), ), - lastNamePreviouslyUsed: v.optional(v.array(v.string())), + lastNamePreviouslyUsed: v.optional(v.array(v.pipe(v.string(), v.trim()))), gender: v.picklist( applicantGenderService.getApplicantGenders().map(({ id }) => id), t('protected:personal-information.gender.required'), @@ -105,9 +105,9 @@ export async function action({ context, request }: Route.ActionArgs) { }) satisfies v.GenericSchema; const input = { - firstNamePreviouslyUsed: formData.getAll('firstNamePreviouslyUsed').map(String), + firstNamePreviouslyUsed: formData.getAll('firstNamePreviouslyUsed').map(String).filter(Boolean), lastNameAtBirth: String(formData.get('lastNameAtBirth')), - lastNamePreviouslyUsed: formData.getAll('lastNamePreviouslyUsed').map(String), + lastNamePreviouslyUsed: formData.getAll('lastNamePreviouslyUsed').map(String).filter(Boolean), gender: String(formData.get('gender')), } satisfies Partial; @@ -243,27 +243,23 @@ export default function PersonalInformation({ actionData, loaderData, params, ma
- {otherFirstNames.map((name) => { - if (name.length > 0) { - return ( -
- {name} - - - -
- ); - } - })} + {otherFirstNames.map((name) => ( +
+ {name} + + + +
+ ))}
- {otherLastNames.map((name) => { - if (name.length > 0) { - return ( -
- {name} - - - -
- ); - } - })} + {otherLastNames.map((name) => ( +
+ {name} + + + +
+ ))}