-
Notifications
You must be signed in to change notification settings - Fork 3
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
feat: complete registration screen #151
Conversation
Preview Firebase Hosting URL: https://mobility-feeds-dev--pr-151-cvxsi2cq.web.app |
await updateProfile(user, { | ||
displayName: values?.fullname, | ||
}); | ||
dispatch(refreshUserInformation(user)); |
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.
This operation should be abstracted from the component. The update of the profile should be in the saga layer not the component.
}); | ||
dispatch(refreshUserInformation(user)); | ||
|
||
navigateTo(ACCOUNT_TARGET); |
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.
The navigation should be done after the network call is succeeded. We need to add error handling in case a network call fails.
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.
LGTM!
Summary:
Closes #148
This PR is only covering the UI components related to the registration process. The logic will need to be implemented as tested as part of the upcoming issues.
Tasks left to completion to ensure proper behaviour of user registration:
CompleteRegistration.tsx
page i.e. Add Captcha to CompleteRegistration and Signup screen #152Expected behavior:
Please make sure these boxes are checked before submitting your pull request - thanks!
./scripts/api-tests.sh
to make sure you didn't break anything