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

register does not work (for emails that are already registered) #325

Closed
zufanka opened this issue Oct 6, 2017 · 4 comments
Closed

register does not work (for emails that are already registered) #325

zufanka opened this issue Oct 6, 2017 · 4 comments

Comments

@zufanka
Copy link
Member

zufanka commented Oct 6, 2017

image

clicking send does not do anything .. if the email is known to the system I guess.
With other email it worked.

#fuckingwithauthentication

@zufanka zufanka added the bug label Oct 6, 2017
@zufanka zufanka changed the title register does not work register does not work (for emails that are already registered) Oct 6, 2017
@nightsh
Copy link
Member

nightsh commented Oct 7, 2017

@georgiana-b I assigned you too to this, as it's part of the "things to keep in mind when designing the authentication backend" series :)

Are we going to fix this one, or leave it to rot in here until we change the backend?

@zufanka
Copy link
Member Author

zufanka commented Nov 19, 2017

ping @georgiana-b
still needs to be open?

@nightsh nightsh added this to the User accounts milestone Dec 1, 2017
@zufanka
Copy link
Member Author

zufanka commented Dec 21, 2017

new authentication fixes this I would believe? @nightsh ?

@georgiana-b
Copy link
Member

Well of course it doesn't. The problem was actually that it didn't give any significant message like "User already exists". These days it does show the correct message and it will also look pretty after #406 so I'll close this.

@ghost ghost removed the to review label Mar 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants