-
Notifications
You must be signed in to change notification settings - Fork 945
Update auth demo to include password validation #7472
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
Conversation
|
Size Report 1Affected ProductsNo changes between base commit (5ef98ff) and merge commit (eca5ba5).Test Logs |
Size Analysis Report 1Affected ProductsNo changes between base commit (5ef98ff) and merge commit (eca5ba5).Test Logs |
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.
Will there be separate PRs to add the password policy check on existing sign in, sign up & reset password flow?
Since password validation is not being implemented as part of these flows, I figured it was best to have a separate section for testing password validation. We could have a button toggle for each of these flows (like enable/disable password validation) with a similar UI to this for displaying the requirements, but I wasn't sure if we wanted to duplicate this UI across these sections or just have one section for demoing password validation. I will add displaying the forceUpgradeOnSignin to the sign-in flow in another PR. |
Update the auth demo to include a section for password validation. As a password is typed in, the status of requirements will be displayed and updated based on what is required by the policy.
See a quick demo at https://drive.google.com/file/d/1WlAkDJrZCS5qiMYSrNLk-LDWjquFPxTA/view?usp=sharing