-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
magento/magento2#19071: Password strength indicator shows No Password… #19073
magento/magento2#19071: Password strength indicator shows No Password… #19073
Conversation
Hi @dimasalamatov. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
@magento-engcom-team give me 2.3-develop instance |
Hi @rogyar. Thank you for your request. I'm working on Magento 2.3-develop instance for you |
Hi @dimasalamatov , please, sign CLA, otherwise, we can't process your pull request |
Hi @rogyar, here is your Magento instance. |
Hi @dimasalamatov. Thank you for the contribution. According to the Code Contributions Guideline (https://devdocs.magento.com/guides/v2.2/contributor-guide/contributing.html#rules) we can accept the change to 2.2-develop branch only if it's present in 2.3-develop branch. |
Hi @rogyar |
Hi @orlangur, thank you for the review. |
Hi @dimasalamatov. Thank you for your contribution. |
… even when a password is entered
Description (*)
The password strength indicator shows "No Password" at times when there is a password entered and "Number of Required Character Classes" option has value "1". There is an situation when zxcvbn() method returns 0 score value and password is valid. In this case displayScore variable will be 0 and as result text "Password Strength: No Password" is visible despite the fact that "Password" field is filled.
Added additional condition that checks if zxcvbn() returns 0 and pass is valid, then displayScore = 1 and
"Password Strength: Weak"
Link to the same PR for 2.3-develop : #19087
Manual testing scenarios (*)
Contribution checklist (*)