Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add new missing detail contact modal #55484

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 19 additions & 0 deletions src/libs/actions/PersonalDetails.ts
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@
key: ONYXKEYS.SESSION,
callback: (val) => {
currentUserEmail = val?.email ?? '';
currentUserAccountID = val?.accountID ?? -1;

Check failure on line 40 in src/libs/actions/PersonalDetails.ts

View workflow job for this annotation

GitHub Actions / Changed files ESLint check

},
});

Expand Down Expand Up @@ -465,6 +465,15 @@
});
}

/**
* Clear errors for the current user's personal details
*/
function clearPersonalDetailsErrors() {
Onyx.merge(ONYXKEYS.PRIVATE_PERSONAL_DETAILS, {
errors: null,
});
}

function updatePersonalDetailsAndShipExpensifyCards(values: FormOnyxValues<typeof ONYXKEYS.FORMS.PERSONAL_DETAILS_FORM>, validateCode: string) {
const parameters: SetPersonalDetailsAndShipExpensifyCardsParams = {
legalFirstName: values.legalFirstName?.trim() ?? '',
Expand Down Expand Up @@ -504,6 +513,15 @@
},
},
],
finallyData: [
{
onyxMethod: Onyx.METHOD.MERGE,
key: ONYXKEYS.PRIVATE_PERSONAL_DETAILS,
value: {
isValidating: false,
},
},
],
});
}

Expand All @@ -523,4 +541,5 @@
updatePronouns,
updateSelectedTimezone,
updatePersonalDetailsAndShipExpensifyCards,
clearPersonalDetailsErrors,
};
Original file line number Diff line number Diff line change
@@ -1,24 +1,22 @@
import React, {useCallback, useMemo, useRef, useState} from 'react';
import type {ForwardedRef} from 'react';
import {View} from 'react-native';
import {useOnyx} from 'react-native-onyx';
import type {OnyxEntry} from 'react-native-onyx';
import HeaderWithBackButton from '@components/HeaderWithBackButton';
import InteractiveStepSubHeader from '@components/InteractiveStepSubHeader';
import type {InteractiveStepSubHeaderHandle} from '@components/InteractiveStepSubHeader';
import ScreenWrapper from '@components/ScreenWrapper';
import ValidateCodeActionModal from '@components/ValidateCodeActionModal';
import useLocalize from '@hooks/useLocalize';
import useSubStep from '@hooks/useSubStep';
import useThemeStyles from '@hooks/useThemeStyles';
import * as FormActions from '@libs/actions/FormActions';

Check failure on line 12 in src/pages/MissingPersonalDetails/MissingPersonalDetailsContent.tsx

View workflow job for this annotation

GitHub Actions / Changed files ESLint check

Namespace imports from @libs are not allowed. Use named imports instead. Example: import { method } from "@libs/module"
import {requestValidateCodeAction} from '@libs/actions/User';
import Navigation from '@libs/Navigation/Navigation';
import * as PersonalDetails from '@userActions/PersonalDetails';
import {updatePersonalDetailsAndShipExpensifyCards} from '@userActions/PersonalDetails';
import CONST from '@src/CONST';
import ONYXKEYS from '@src/ONYXKEYS';
import type {PersonalDetailsForm} from '@src/types/form';
import type {PrivatePersonalDetails} from '@src/types/onyx';
import MissingPersonalDetailsMagicCodeModal from './MissingPersonalDetailsMagicCodeModal';
import Address from './substeps/Address';
import Confirmation from './substeps/Confirmation';
import DateOfBirth from './substeps/DateOfBirth';
Expand All @@ -37,9 +35,6 @@
function MissingPersonalDetailsContent({privatePersonalDetails, draftValues}: MissingPersonalDetailsContentProps) {
const styles = useThemeStyles();
const {translate} = useLocalize();
const [account] = useOnyx(ONYXKEYS.ACCOUNT);
const [validateCodeAction] = useOnyx(ONYXKEYS.VALIDATE_ACTION_CODE);
const primaryLogin = account?.primaryLogin ?? '';
const [isValidateCodeActionModalVisible, setIsValidateCodeActionModalVisible] = useState(false);

const ref: ForwardedRef<InteractiveStepSubHeaderHandle> = useRef(null);
Expand Down Expand Up @@ -81,23 +76,13 @@
prevScreen();
};

const handleValidateCodeEntered = useCallback(
const handleSubmitForm = useCallback(
(validateCode: string) => {
PersonalDetails.updatePersonalDetailsAndShipExpensifyCards(values, validateCode);
FormActions.clearDraftValues(ONYXKEYS.FORMS.PERSONAL_DETAILS_FORM);
Navigation.goBack();
updatePersonalDetailsAndShipExpensifyCards(values, validateCode);
},
[values],
);

const sendValidateCode = () => {
if (validateCodeAction?.validateCodeSent) {
return;
}

requestValidateCodeAction();
};

const handleNextScreen = useCallback(() => {
if (isEditing) {
goToTheLastStep();
Expand Down Expand Up @@ -132,15 +117,10 @@
personalDetailsValues={values}
/>

<ValidateCodeActionModal
handleSubmitForm={handleValidateCodeEntered}
sendValidateCode={sendValidateCode}
clearError={() => {}}
<MissingPersonalDetailsMagicCodeModal
onClose={() => setIsValidateCodeActionModalVisible(false)}
isVisible={isValidateCodeActionModalVisible}
title={translate('cardPage.validateCardTitle')}
descriptionPrimary={translate('cardPage.enterMagicCode', {contactMethod: primaryLogin})}
hasMagicCodeBeenSent={!!validateCodeAction?.validateCodeSent}
isValidateCodeActionModalVisible={isValidateCodeActionModalVisible}
handleSubmitForm={handleSubmitForm}
/>
</ScreenWrapper>
);
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,81 @@
import React, {useEffect} from 'react';
import {useOnyx} from 'react-native-onyx';
import ValidateCodeActionModal from '@components/ValidateCodeActionModal';
import useLocalize from '@hooks/useLocalize';
import * as FormActions from '@libs/actions/FormActions';

Check failure on line 5 in src/pages/MissingPersonalDetails/MissingPersonalDetailsMagicCodeModal.tsx

View workflow job for this annotation

GitHub Actions / Changed files ESLint check

Namespace imports from @libs are not allowed. Use named imports instead. Example: import { method } from "@libs/module"
import {clearPersonalDetailsErrors} from '@libs/actions/PersonalDetails';
import {requestValidateCodeAction} from '@libs/actions/User';
import * as ErrorUtils from '@libs/ErrorUtils';

Check failure on line 8 in src/pages/MissingPersonalDetails/MissingPersonalDetailsMagicCodeModal.tsx

View workflow job for this annotation

GitHub Actions / Changed files ESLint check

Namespace imports from @libs are not allowed. Use named imports instead. Example: import { method } from "@libs/module"
import Navigation from '@libs/Navigation/Navigation';
import ONYXKEYS from '@src/ONYXKEYS';
import {isEmptyObject} from '@src/types/utils/EmptyObject';

type MissingPersonalDetailsMagicCodeModalProps = {
handleSubmitForm: (validateCode: string) => void;
isValidateCodeActionModalVisible: boolean;
onClose?: () => void;
};

function MissingPersonalDetailsMagicCodeModal({onClose, isValidateCodeActionModalVisible, handleSubmitForm}: MissingPersonalDetailsMagicCodeModalProps) {
const {translate} = useLocalize();
const [privatePersonalDetails] = useOnyx(ONYXKEYS.PRIVATE_PERSONAL_DETAILS);
const [account] = useOnyx(ONYXKEYS.ACCOUNT);
const [validateCodeAction] = useOnyx(ONYXKEYS.VALIDATE_ACTION_CODE);
const privateDetailsErrors = privatePersonalDetails?.errors ?? undefined;
const validateLoginError = ErrorUtils.getLatestError(privateDetailsErrors);
const primaryLogin = account?.primaryLogin ?? '';

const missingDetails =
!privatePersonalDetails?.legalFirstName ||
!privatePersonalDetails?.legalLastName ||
!privatePersonalDetails?.dob ||
!privatePersonalDetails?.phoneNumber ||
isEmptyObject(privatePersonalDetails?.addresses) ||
privatePersonalDetails.addresses.length === 0;

useEffect(() => {
if (missingDetails || !!privateDetailsErrors || privatePersonalDetails?.isValidating) {
return;
}

FormActions.clearDraftValues(ONYXKEYS.FORMS.PERSONAL_DETAILS_FORM);
Navigation.goBack();
}, [missingDetails, privateDetailsErrors, privatePersonalDetails?.isValidating]);

const sendValidateCode = () => {
if (validateCodeAction?.validateCodeSent) {
return;
}

requestValidateCodeAction();
};

const onBackButtonPress = () => {
onClose?.();
};

const clearError = () => {
if (!validateLoginError) {
return;
}
clearPersonalDetailsErrors();
};

return (
<ValidateCodeActionModal
clearError={clearError}
onClose={onBackButtonPress}
validateError={validateLoginError}
isVisible={isValidateCodeActionModalVisible}
title={translate('cardPage.validateCardTitle')}
descriptionPrimary={translate('cardPage.enterMagicCode', {contactMethod: primaryLogin})}
sendValidateCode={sendValidateCode}
hasMagicCodeBeenSent={validateCodeAction?.validateCodeSent}
handleSubmitForm={handleSubmitForm}
/>
);
}

MissingPersonalDetailsMagicCodeModal.displayName = 'MissingPersonalDetailsMagicCodeModal';

export default MissingPersonalDetailsMagicCodeModal;
6 changes: 6 additions & 0 deletions src/types/onyx/PrivatePersonalDetails.ts
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,12 @@ type PrivatePersonalDetails = {

/** Error objects keyed by field name containing errors keyed by microtime */
errorFields?: OnyxCommon.ErrorFields;

/** Authentication failure errors */
errors?: OnyxCommon.Errors | null;

/** Whether the API is loading */
isValidating?: boolean;
};

export default PrivatePersonalDetails;
Expand Down
Loading