Merge pull request #324 from dod-ccpo/fix-members-list-screen

Use proper display name for workspace roles
This commit is contained in:
richard-dds
2018-09-25 10:12:57 -04:00
committed by GitHub
8 changed files with 86 additions and 7 deletions

View File

@@ -234,17 +234,19 @@ def update_member(workspace_id, member_id):
form = EditMemberForm(http_request.form)
if form.validate():
role = None
new_role_name = None
if form.data["workspace_role"] != member.role:
role = form.data["workspace_role"]
Workspaces.update_member(g.current_user, workspace, member, role)
member = Workspaces.update_member(
g.current_user, workspace, member, form.data["workspace_role"]
)
new_role_name = member.role_displayname
return redirect(
url_for(
"workspaces.workspace_members",
workspace_id=workspace.id,
memberName=member.user_name,
updatedRole=role,
updatedRole=new_role_name,
)
)
else: