-
-
Notifications
You must be signed in to change notification settings - Fork 115
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
Username Validation Incorrect Warning #718
Comments
@daniel-n-kaye thank you for suggesting this! @all-contributors add @daniel-n-kaye for idea |
I couldn't determine any contributions to add, did you specify any contributions? I've put up a pull request to add @daniel-n-kaye! 🎉 |
@vnugent I'd be willing to try to take this on as a first issue, if that's ok, as it seems like it would hopefully be a pretty isolated update. |
The error messages are listed here
Since profile update code requires authentication with Auth0, please send an email: viet at openbeta dot io and I'll reply with some additional API keys for the dev environment. |
@vnugent I think my pull request will do the work. |
Is this still active? |
This is still an active issue. @danekshea any updates on your changes? |
I'm working on moving some of user profile functionality to our own DB. Username update is the first piece so I can take on this issue. What do you all think the error message should be? Username validation rules:
See #830 |
Based on my understanding of the regex, it appears that it requires a username to begin and end with a letter or number. It cannot contain spaces but it can contain "." or "-" or "_" Would these be considered special characters? I was brainstorming on how best to write this error message so that users can understand it clearly. What about:
It's verbose but it is specific. @zichongkao had some ideas. That might be a long error message....we could break the regex into components so that it would trip each test as the User tried their new nickname. |
dot and underscore are allowed (but not as the first character) |
I'm okay with it. |
This issue tracker is ONLY used for reporting bugs!
Expected Behavior
When editing your profile user name, you get an error when there are spaces in the user name. The warning message should probably say 'username cannot contain spaces' or something.
Current Behavior
The actual error message says 'Must start and end with a letter or number' which I think is incorrect
Possible Solution
Show correct user name validation warning, maybe something like "User name cannot contain spaces" or "User name must contain only letters and numbers"
The text was updated successfully, but these errors were encountered: