Skip to content

Commit

Permalink
fix lint
Browse files Browse the repository at this point in the history
  • Loading branch information
daledah committed Feb 5, 2025
1 parent 1c7a4e3 commit 9748196
Show file tree
Hide file tree
Showing 4 changed files with 19 additions and 14 deletions.
8 changes: 4 additions & 4 deletions src/components/SubStepForms/DateOfBirthStep.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ import Text from '@components/Text';
import useLocalize from '@hooks/useLocalize';
import type {SubStepProps} from '@hooks/useSubStep/types';
import useThemeStyles from '@hooks/useThemeStyles';
import * as ValidationUtils from '@libs/ValidationUtils';
import {getFieldRequiredErrors, isValidPastDate, meetsMaximumAgeRequirement, meetsMinimumAgeRequirement} from '@libs/ValidationUtils';
import CONST from '@src/CONST';
import type {OnyxFormValuesMapping} from '@src/ONYXKEYS';

Expand Down Expand Up @@ -57,14 +57,14 @@ function DateOfBirthStep<TFormID extends keyof OnyxFormValuesMapping>({

const validate = useCallback(
(values: FormOnyxValues<TFormID>): FormInputErrors<TFormID> => {
const errors = ValidationUtils.getFieldRequiredErrors(values, stepFields);
const errors = getFieldRequiredErrors(values, stepFields);

const valuesToValidate = values[dobInputID as keyof FormOnyxValues<TFormID>] as string;
if (valuesToValidate) {
if (!ValidationUtils.isValidPastDate(valuesToValidate) || !ValidationUtils.meetsMaximumAgeRequirement(valuesToValidate)) {
if (!isValidPastDate(valuesToValidate) || !meetsMaximumAgeRequirement(valuesToValidate)) {
// @ts-expect-error type mismatch to be fixed
errors[dobInputID] = translate('bankAccount.error.dob');
} else if (!ValidationUtils.meetsMinimumAgeRequirement(valuesToValidate)) {
} else if (!meetsMinimumAgeRequirement(valuesToValidate)) {
// @ts-expect-error type mismatch to be fixed
errors[dobInputID] = translate('bankAccount.error.age');
}
Expand Down
12 changes: 6 additions & 6 deletions src/components/SubStepForms/FullNameStep.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ import TextInput from '@components/TextInput';
import useLocalize from '@hooks/useLocalize';
import type {SubStepProps} from '@hooks/useSubStep/types';
import useThemeStyles from '@hooks/useThemeStyles';
import * as ValidationUtils from '@libs/ValidationUtils';
import {getFieldRequiredErrors, isRequiredFulfilled, isValidLegalName} from '@libs/ValidationUtils';
import HelpLinks from '@pages/ReimbursementAccount/PersonalInfo/HelpLinks';
import CONST from '@src/CONST';
import type {OnyxFormValuesMapping} from '@src/ONYXKEYS';
Expand Down Expand Up @@ -70,13 +70,13 @@ function FullNameStep<TFormID extends keyof OnyxFormValuesMapping>({

const validate = useCallback(
(values: FormOnyxValues<TFormID>): FormInputErrors<TFormID> => {
const errors = ValidationUtils.getFieldRequiredErrors(values, stepFields);
const errors = getFieldRequiredErrors(values, stepFields);

const firstName = values[firstNameInputID as keyof FormOnyxValues<TFormID>] as string;
if (!ValidationUtils.isRequiredFulfilled(firstName)) {
if (!isRequiredFulfilled(firstName)) {
// @ts-expect-error type mismatch to be fixed
errors[firstNameInputID] = translate('common.error.fieldRequired');
} else if (!ValidationUtils.isValidLegalName(firstName)) {
} else if (!isValidLegalName(firstName)) {
// @ts-expect-error type mismatch to be fixed
errors[firstNameInputID] = translate('privatePersonalDetails.error.hasInvalidCharacter');
} else if (firstName.length > CONST.LEGAL_NAME.MAX_LENGTH) {
Expand All @@ -88,10 +88,10 @@ function FullNameStep<TFormID extends keyof OnyxFormValuesMapping>({
}

const lastName = values[lastNameInputID as keyof FormOnyxValues<TFormID>] as string;
if (!ValidationUtils.isRequiredFulfilled(lastName)) {
if (!isRequiredFulfilled(lastName)) {
// @ts-expect-error type mismatch to be fixed
errors[lastNameInputID] = translate('common.error.fieldRequired');
} else if (!ValidationUtils.isValidLegalName(lastName)) {
} else if (!isValidLegalName(lastName)) {
// @ts-expect-error type mismatch to be fixed
errors[lastNameInputID] = translate('privatePersonalDetails.error.hasInvalidCharacter');
} else if (lastName.length > CONST.LEGAL_NAME.MAX_LENGTH) {
Expand Down
6 changes: 5 additions & 1 deletion src/components/SubStepForms/SingleFieldStep.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,9 @@ type SingleFieldStepProps<TFormID extends keyof OnyxFormValuesMapping> = SubStep

/** Max length of the field */
maxLength?: number;

/** Should the submit button be enabled when offline */
enabledWhenOffline?: boolean;
};

function SingleFieldStep<TFormID extends keyof OnyxFormValuesMapping>({
Expand All @@ -61,6 +64,7 @@ function SingleFieldStep<TFormID extends keyof OnyxFormValuesMapping>({
isEditing,
shouldShowHelpLinks = true,
maxLength,
enabledWhenOffline,
}: SingleFieldStepProps<TFormID>) {
const {translate} = useLocalize();
const styles = useThemeStyles();
Expand All @@ -73,7 +77,7 @@ function SingleFieldStep<TFormID extends keyof OnyxFormValuesMapping>({
onSubmit={onSubmit}
style={[styles.mh5, styles.flexGrow1]}
submitButtonStyles={[styles.mb0]}
enabledWhenOffline
enabledWhenOffline={enabledWhenOffline}
>
<View>
<Text style={[styles.textHeadlineLineHeightXXL]}>{formTitle}</Text>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ import SingleFieldStep from '@components/SubStepForms/SingleFieldStep';
import useLocalize from '@hooks/useLocalize';
import type {SubStepProps} from '@hooks/useSubStep/types';
import useWalletAdditionalDetailsStepFormSubmit from '@hooks/useWalletAdditionalDetailsStepFormSubmit';
import * as ValidationUtils from '@libs/ValidationUtils';
import {getFieldRequiredErrors, isValidUSPhone} from '@libs/ValidationUtils';
import CONST from '@src/CONST';
import ONYXKEYS from '@src/ONYXKEYS';
import INPUT_IDS from '@src/types/form/WalletAdditionalDetailsForm';
Expand All @@ -21,9 +21,9 @@ function PhoneNumberStep({onNext, onMove, isEditing}: SubStepProps) {

const validate = useCallback(
(values: FormOnyxValues<typeof ONYXKEYS.FORMS.WALLET_ADDITIONAL_DETAILS>): FormInputErrors<typeof ONYXKEYS.FORMS.WALLET_ADDITIONAL_DETAILS> => {
const errors = ValidationUtils.getFieldRequiredErrors(values, STEP_FIELDS);
const errors = getFieldRequiredErrors(values, STEP_FIELDS);

if (values.phoneNumber && !ValidationUtils.isValidUSPhone(values.phoneNumber, true)) {
if (values.phoneNumber && !isValidUSPhone(values.phoneNumber, true)) {
errors.phoneNumber = translate('bankAccount.error.phoneNumber');
}
return errors;
Expand Down Expand Up @@ -51,6 +51,7 @@ function PhoneNumberStep({onNext, onMove, isEditing}: SubStepProps) {
inputLabel={translate('common.phoneNumber')}
inputMode={CONST.INPUT_MODE.TEL}
defaultValue={defaultPhoneNumber}
enabledWhenOffline
/>
);
}
Expand Down

0 comments on commit 9748196

Please sign in to comment.