-
Notifications
You must be signed in to change notification settings - Fork 945
Include the character option statuses even when an empty string is inputted to validatePassword #7471
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
|
// Assign statuses for requirements even if the password is an empty string. | ||
this.updatePasswordCharacterOptionsStasuses( | ||
status, | ||
/* containsLowercaseCharacter= */ false, |
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.
I added parameter name comments to make this more readable as per the TS/JS style guide.
Size Report 1Affected Products
Test Logs |
Size Analysis Report 1Affected Products
Test Logs |
…putted to validatePassword (#7471) * Include the character option statuses even when an empty string is inputted
Fix validatePasswordCharacterOptions to include the character options statuses even when an empty string is inputted.