darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-07-26 15:48:28 -04:00
Code Issues Packages Projects Releases Wiki Activity

Revert "replace user PUT with POSTs"

Browse Source
This commit is contained in:
Leo Hemsted
2017-11-09 14:57:01 +00:00
committed by GitHub
parent 395a2fccbb
commit bfa6980913
7 changed files with 27 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
app/notify_client/user_api_client.py
View File

@@ -54,6 +54,12 @@ class UserApiClient(NotifyAdminAPIClient):
users.append(User(user, max_failed_login_count=self.max_failed_login_count))
return users
def update_user(self, user):
data = user.serialize()
url = "/user/{}".format(user.id)
user_data = self.put(url, data=data)
return User(user_data['data'], max_failed_login_count=self.max_failed_login_count)
def update_user_attribute(self, user_id, **kwargs):
data = dict(kwargs)
disallowed_attributes = set(data.keys()) - ALLOWED_ATTRIBUTES
@@ -149,9 +155,7 @@ class UserApiClient(NotifyAdminAPIClient):
def activate_user(self, user):
if user.state == 'pending':
user.state = 'active'
url = "/user/{}".format(user.id)
user_data = self.post(url, data={'state': 'active'})
return User(user_data['data'], max_failed_login_count=self.max_failed_login_count)
return self.update_user(user)
else:
return user