Skip to content
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

Password error msg #2113

Closed
wants to merge 2 commits into from
Closed

Password error msg #2113

wants to merge 2 commits into from

Conversation

triplegem
Copy link

What?

Password validation messages upon new account creation are not in sync. During password character inputting, a message appears stating the password must be 7 characters long with numbers and letters.

During password validation upon Submit, the message is different: it states the password must be 8 characters long with lower case letters, upper case letters, a number and a special character.

Requirements

  • CHANGELOG.md entry added (required for code changes only)

Tickets / Documentation

Add links to any relevant tickets and documentation.

Screenshots (if appropriate)

BC-password-validation

Example Image

@bigbot
Copy link

bigbot commented Sep 2, 2021

Autotagging @bigcommerce/themes-team

@bc-alexsaiannyi
Copy link
Contributor

Hi @cl939 and thanks for your contribution!

I believe this issue will gone with merging PR#2117. You can check our discussion there and details on the invalid password error text.

For this reason I think we can close this PR. Thank you again!

FYI @yurytut1993 @BC-krasnoshapka

@bc-yevhenii-buliuk
Copy link
Contributor

This PR is closed due to the solution of this issue by another PR (#2117)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants