-
Notifications
You must be signed in to change notification settings - Fork 70
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
[api] /users API - v0.1.0 #5
Comments
are we handling 422(unprocessable entity) here? need to add backend validations - username should be in email format, password should be more than 6 and both fields should be mandatory |
and email should be unique |
some of these I can see on PR already |
@amsanket22 yes all proper HTTP codes. 409 for conflict (unique email violation), and 422 for unprocessable (any parsing error in the body), 400 for everything else |
this is done now |
Subtasks
The text was updated successfully, but these errors were encountered: