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

User information fields are not verified #19

Open
pekman opened this issue Nov 19, 2010 · 2 comments
Open

User information fields are not verified #19

pekman opened this issue Nov 19, 2010 · 2 comments
Assignees

Comments

@pekman
Copy link

pekman commented Nov 19, 2010

When editing user information, ASI doesn't allow email and gender fields to be changed to empty. There should be a local verification for these values and error messages should be shown to the user, if the values are invalid.

@ghost ghost assigned pekman May 9, 2011
@pekman
Copy link
Author

pekman commented Jun 7, 2011

Current behavior: If these fields are empty on submit, they are silently ignored and the corresponding values in ASI remain unchanged.

It might be better if empty email displayed an error message. The status of the gender field is more complicated: When creating a new user, ASI allows gender to be unspecified, but once it's been set to male or female, ASI doesn't allow it to be changed to unspecified. It might be best to just require the users to always specify their gender.

pekman pushed a commit that referenced this issue Jun 7, 2011
@pekman
Copy link
Author

pekman commented Jun 7, 2011

Email field fixed. The issue with gender field remains. As an additional complication, there is no gender field when creating a new user and adding the field requires an additional ASI request on creating new users.

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

1 participant