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

Registration page should display error message if user tries to register a duplicate account #1127

Closed
rstorey opened this issue Oct 16, 2019 · 6 comments · Fixed by #1134
Closed
Assignees
Labels
bug Something isn't working onboarding

Comments

@rstorey
Copy link
Member

rstorey commented Oct 16, 2019

What behavior did you observe? Please describe the bug
The register page refreshed. The password and confirm password fields emptied.

How can we reproduce the bug?
Steps to reproduce the behavior:

  1. Go to the Register page
  2. Fill out the form and submit.
  3. Go back to a blank Register page.
  4. Fill out the form using the same username or email address as in the first try

What is the expected behavior?
If a user attempts to register either a username or email address that is already in use by another account, an error message should be displayed and the user should be prompted to use a different username / email address. Perhaps there should also be a link to the reset password page.

Desktop (please complete the following information):

  • OS: MacOS
  • Browser Chrome
@acdha
Copy link
Member

acdha commented Oct 16, 2019

The password fields employing is a security measure. This used to work and the form sets the invalid state on the email field so this was probably a template change suppressing display of the error message for that field since we chose not to use the standard Django forms.

@rstorey
Copy link
Member Author

rstorey commented Oct 16, 2019

Yep. I only mentioned the password thing because that was the only way I could tell the page actually refreshed.

@rstorey rstorey added onboarding bug Something isn't working labels Oct 17, 2019
@rstorey rstorey self-assigned this Oct 17, 2019
@rstorey
Copy link
Member Author

rstorey commented Oct 29, 2019

@rstorey
Copy link
Member Author

rstorey commented Oct 29, 2019

@rstorey
Copy link
Member Author

rstorey commented Oct 29, 2019

twbs/bootstrap#23454

@rstorey
Copy link
Member Author

rstorey commented Oct 29, 2019

twbs/bootstrap#29439

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working onboarding
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

2 participants