Skip to content

fix: Sign Up not working #1889

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

Merged
merged 1 commit into from
Jun 3, 2019

Conversation

liveHarshit
Copy link
Member

Fixes #1870

  • According to the server implementation, password length should be 8 characters.

@auto-label auto-label bot added the fix label Jun 3, 2019
@ShridharGoel
Copy link
Member

Shouldn't the error message be changed since it specifically says that the email is invalid while error 422 can be for other reasons too?

@liveHarshit
Copy link
Member Author

Shouldn't the error message be changed since it specifically says that the email is invalid while error 422 can be for other reasons too?

Yes. @nikit19 Where can I find docs about these error messages, because these are not in the server docs?

@anhanh11001
Copy link
Contributor

So what was the reason for sign up not working properly (just because of short password?)

@vartult
Copy link
Member

vartult commented Jun 3, 2019

So what was the reason for sign up not working properly (just because of short password?)

Yes the required minimum length for password is 8.

@iamareebjamal iamareebjamal merged commit 2b39b06 into fossasia:development Jun 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Authentication fragments e-mail validation not working.
5 participants