Skip to content

Commit 9d6e40c

Browse files
committed
get stored phonenumber from privatePersonalDetails
1 parent f881f56 commit 9d6e40c

3 files changed

Lines changed: 6 additions & 3 deletions

File tree

src/pages/EnablePayments/Pay/AdditionalDetails/AdditionalDetailsStep.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ function AdditionalDetailsStep({currentUserPersonalDetails}: AdditionalDetailsSt
135135
dob: values.dob ?? '',
136136
ssn: values.ssn ?? '',
137137
};
138-
submitPersonalDetails(personalDetails, currentUserPersonalDetails.phoneNumber);
138+
submitPersonalDetails(personalDetails);
139139
};
140140

141141
if (isMagicCodeRequired) {

src/pages/EnablePayments/Wallet/PersonalInfo/PersonalInfo.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,7 @@ function PersonalInfoPage() {
5555
const {isMagicCodeRequired, submitPersonalDetails, confirmPersonalDetailsWithMagicCode, closeMagicCodePrompt} = useWalletPhoneMagicCode();
5656

5757
const values = useMemo(() => getSubstepValues(PERSONAL_INFO_STEP_KEYS, walletAdditionalDetailsDraft, walletAdditionalDetails), [walletAdditionalDetails, walletAdditionalDetailsDraft]);
58+
5859
const submit = () => {
5960
const personalDetails: UpdatePersonalDetailsForWalletParams = {
6061
phoneNumber: (values.phoneNumber && parsePhoneNumber(values.phoneNumber, {regionCode: CONST.COUNTRY.US}).number?.significant) ?? '',
@@ -67,7 +68,7 @@ function PersonalInfoPage() {
6768
dob: values?.[PERSONAL_INFO_STEP_KEYS.DOB] ?? '',
6869
ssn: values?.[PERSONAL_INFO_STEP_KEYS.SSN_LAST_4] ?? '',
6970
};
70-
submitPersonalDetails(personalDetails, walletAdditionalDetails?.phoneNumber);
71+
submitPersonalDetails(personalDetails);
7172
};
7273

7374
const startFrom = useMemo(() => getInitialSubstepForPersonalInfo(values), [values]);

src/pages/EnablePayments/shared/useWalletPhoneMagicCode.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@ import {useEffect, useRef, useState} from 'react';
1818
function useWalletPhoneMagicCode() {
1919
const [walletAdditionalDetails] = useOnyx(ONYXKEYS.WALLET_ADDITIONAL_DETAILS);
2020
const [formData] = useOnyx(ONYXKEYS.FORMS.WALLET_ADDITIONAL_DETAILS);
21+
const [privatePersonalDetails] = useOnyx(ONYXKEYS.PRIVATE_PERSONAL_DETAILS);
2122

2223
// The details the user submitted, held while we prompt for a magic code to confirm a phone number change
2324
const submittedPersonalDetailsRef = useRef<UpdatePersonalDetailsForWalletParams | null>(null);
@@ -43,11 +44,12 @@ function useWalletPhoneMagicCode() {
4344
}, [formData?.isLoading, walletAdditionalDetails?.errorCode]);
4445

4546
// Submits the personal details, first prompting for a magic code when an existing phone number is being changed.
46-
const submitPersonalDetails = (personalDetails: UpdatePersonalDetailsForWalletParams, storedPhoneNumber: string | undefined) => {
47+
const submitPersonalDetails = (personalDetails: UpdatePersonalDetailsForWalletParams) => {
4748
submittedPersonalDetailsRef.current = personalDetails;
4849

4950
// The stored phone number keeps its country code, so normalize it the same way as the submitted one before
5051
// comparing, otherwise an unchanged phone would look like a change and wrongly prompt for a magic code.
52+
const storedPhoneNumber = privatePersonalDetails?.phoneNumber;
5153
const normalizedStoredPhoneNumber = (storedPhoneNumber && parsePhoneNumber(storedPhoneNumber, {regionCode: CONST.COUNTRY.US}).number?.significant) ?? '';
5254
const hasPhoneNumberChanged = !!normalizedStoredPhoneNumber && personalDetails.phoneNumber !== normalizedStoredPhoneNumber;
5355
if (hasPhoneNumberChanged) {

0 commit comments

Comments
 (0)