darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-05-03 23:51:22 -04:00
Code Issues Packages Projects Releases Wiki Activity

ensure invited user permissions show up correctly

Browse Source
This commit is contained in:
Leo Hemsted
2018-03-01 16:59:01 +00:00
parent 7f268c0ab3
commit 793d79c242
4 changed files with 8 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
app/notify_client/user_api_client.py
View File

@@ -148,8 +148,7 @@ class UserApiClient(NotifyAdminAPIClient):
# permissions passed in are the combined admin roles, not db permissions
endpoint = '/service/{}/users/{}'.format(service_id, user_id)
data = [{'permission': x} for x in translate_permissions_from_admin_roles_to_db(permissions)]
resp = self.post(endpoint, data=data)
return User(resp['data'], max_failed_login_count=self.max_failed_login_count)
self.post(endpoint, data=data)
def add_user_to_organisation(self, org_id, user_id):
resp = self.post('/organisations/{}/users/{}'.format(org_id, user_id), data={})