Slight changes to match new API features:

- get user now throws a 404 so handle that
- making a service now needs one user not many to create
This commit is contained in:
Martyn Inglis
2016-02-19 16:38:08 +00:00
parent f761afa76e
commit 6616182ab3
3 changed files with 11 additions and 5 deletions

View File

@@ -1,4 +1,5 @@
from datetime import datetime
from notifications_python_client import HTTPError
from sqlalchemy.orm import load_only
@@ -47,9 +48,15 @@ def activate_user(user):
def is_email_unique(email_address):
if user_api_client.get_user_by_email(email_address):
return False
return True
try:
if user_api_client.get_user_by_email(email_address):
return False
return True
except HTTPError as ex:
if ex.status_code == 404:
return True
else:
raise ex
def request_password_reset(user):

View File

@@ -26,7 +26,6 @@ def register():
return redirect(url_for('main.choose_service'))
form = RegisterUserForm()
if form.validate_on_submit():
if users_dao.is_email_unique(form.email_address.data):
try:

View File

@@ -24,7 +24,7 @@ class NotificationsAdminAPIClient(NotificationsAPIClient):
"name": service_name,
"active": active,
"limit": limit,
"users": [user_id],
"user_id": user_id,
"restricted": restricted
}
return self.post("/service", data)