-
Notifications
You must be signed in to change notification settings - Fork 0
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
Suisin/upm826/success modal phone number verification #89
Suisin/upm826/success modal phone number verification #89
Conversation
packages/stores/types.ts
Outdated
@@ -744,6 +744,8 @@ type TUiStore = { | |||
setRealAccountSignupEnd: (status: boolean) => void; | |||
setPurchaseState: (index: number) => void; | |||
simple_duration_unit: string; | |||
should_show_phone_number_verified_modal: boolean; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we need this observer ?
This modal will be triggered only when phone number otp is verified. Isnt?
Can we just trigger the modal on success on API call then?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeap but the modal will be displayed on personal-details-page, so when we check for verified: 1 api call this modal will show up everytime isn't 🤔 I was thinking that when user clicks on verify
button, when the API call isSuccess then we put the value in store and redirect user back to personal details then dispaly the successful modal
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lets check with design team on this flow
...account/src/Sections/Profile/PhoneVerification/__test__/phone-number-verified-modal.spec.tsx
Outdated
Show resolved
Hide resolved
43cb445
into
suisin/utkarsha/UPM819/phone_verification_flow
Changes:
Add success modal for phone number verification
Screenshots: