Skip to content

Commit

Permalink
Fix phone taken user error
Browse files Browse the repository at this point in the history
  • Loading branch information
sokolovstas committed Feb 3, 2022
1 parent 61a5c67 commit 83b6a75
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
3 changes: 3 additions & 0 deletions web/api/messages.go
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ var messages = map[MessageID]string{
ErrorAPIUsernameTaken: "Username is taken. Try to choose another one",
ErrorAPIUsernameEmailOrPhoneTaken: "Username, email or/and phone is taken. Try to choose another one",
ErrorAPIEmailTaken: "Email is taken. Try to choose another one",
ErrorAPIPhoneTaken: "Phone is taken. Try to choose another one",
ErrorAPIInviteTokenServerError: "Unable to create invite token. Try again or contact support team",
ErrorAPIInviteUnableToInvalidate: "Unable to invalidate invite. Try again or contact support team",
ErrorAPIInviteUnableToSave: "Unable to save invite. Try again or contact support team",
Expand Down Expand Up @@ -71,6 +72,8 @@ const (
ErrorAPIUsernameEmailOrPhoneTaken = "error.api.username_phone_email.taken"
// ErrorAPIEmailTaken is when email is already taken.
ErrorAPIEmailTaken = "error.api.email.taken"
// ErrorAPIPhoneTaken is when phone is already taken.
ErrorAPIPhoneTaken = "error.api.phone.taken"
// ErrorAPIInviteTokenServerError is for invite token creation issues.
ErrorAPIInviteTokenServerError = "error.api.invite_token.server_error"
// ErrorAPIInviteUnableToInvalidate is when invite cannot be invalidated.
Expand Down
2 changes: 1 addition & 1 deletion web/api/update_user.go
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ func (ar *Router) UpdateUser() http.HandlerFunc {
// Check that phone is not taken.
if d.updatePhone {
if _, err := ar.server.Storages().User.UserByPhone(d.NewPhone); err == nil {
ar.Error(w, ErrorAPIEmailTaken, http.StatusBadRequest, "", "UpdateUser.updatePhone && UserByPhone")
ar.Error(w, ErrorAPIPhoneTaken, http.StatusBadRequest, "", "UpdateUser.updatePhone && UserByPhone")
return
}
}
Expand Down

0 comments on commit 83b6a75

Please sign in to comment.