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

Full Name validation needs to be improved #1250

Open
hinakhadim opened this issue May 30, 2024 · 4 comments
Open

Full Name validation needs to be improved #1250

hinakhadim opened this issue May 30, 2024 · 4 comments
Assignees

Comments

@hinakhadim
Copy link
Contributor

hinakhadim commented May 30, 2024

When registering as a user in authn MFE Register tab, there is a full Name field.

  • That field allows user to enter single character. I believe that the validation should update and must allow to enter at least 3 characters.
  • The full name field is accepting the special characters as well. Full name doesn't contain any special characters so validation should include only alphabets

image


image

@jciasenza
Copy link

I think it's fine that you accept special characters because in different languages ​​there are characters that are used and are not recognized as alphabetic and neither is the quantity.
But if it needs to be reviewed I could correct the Issue. @arbrandes

@jciasenza
Copy link

Hi hinakhadim, I made PR #1348, please review it.

@brian-smith-tcril
Copy link
Contributor

@ormsbee
Copy link

ormsbee commented Dec 10, 2024

I made a comment in the above referenced PR explaining why I am skeptical about stricter name validation. If you agree with that assessment, please close the PR and this issue. If you would still like to pursue stricter validation through the product review process, you are welcome to do so and keep this issue open. Please just leave a comment in this ticket indicating your intentions.

Thank you.

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

No branches or pull requests

4 participants