Standardize member and env name ordering
This commit is contained in:
parent
3d92ac4840
commit
a8d5201cc6
@ -25,6 +25,7 @@ class Application(
|
|||||||
primaryjoin=and_(
|
primaryjoin=and_(
|
||||||
Environment.application_id == id, Environment.deleted == False
|
Environment.application_id == id, Environment.deleted == False
|
||||||
),
|
),
|
||||||
|
order_by="Environment.name",
|
||||||
)
|
)
|
||||||
roles = relationship(
|
roles = relationship(
|
||||||
"ApplicationRole",
|
"ApplicationRole",
|
||||||
|
@ -30,9 +30,9 @@ def get_environments_obj_for_app(application):
|
|||||||
"pending": env.is_pending,
|
"pending": env.is_pending,
|
||||||
"edit_form": EditEnvironmentForm(obj=env),
|
"edit_form": EditEnvironmentForm(obj=env),
|
||||||
"member_count": len(env.roles),
|
"member_count": len(env.roles),
|
||||||
"members": [
|
"members": sorted(
|
||||||
env_role.application_role.user_name for env_role in env.roles
|
[env_role.application_role.user_name for env_role in env.roles]
|
||||||
],
|
),
|
||||||
}
|
}
|
||||||
for env in application.environments
|
for env in application.environments
|
||||||
],
|
],
|
||||||
@ -100,12 +100,11 @@ def get_members_data(application):
|
|||||||
form = UpdateMemberForm(
|
form = UpdateMemberForm(
|
||||||
environment_roles=env_roles_form_data, **permission_sets
|
environment_roles=env_roles_form_data, **permission_sets
|
||||||
)
|
)
|
||||||
update_invite_form = None
|
update_invite_form = (
|
||||||
|
MemberForm(obj=member.latest_invitation)
|
||||||
if member.latest_invitation and member.latest_invitation.can_resend:
|
if member.latest_invitation and member.latest_invitation.can_resend
|
||||||
update_invite_form = MemberForm(obj=member.latest_invitation)
|
else MemberForm()
|
||||||
else:
|
)
|
||||||
update_invite_form = MemberForm()
|
|
||||||
|
|
||||||
members_data.append(
|
members_data.append(
|
||||||
{
|
{
|
||||||
@ -119,14 +118,17 @@ def get_members_data(application):
|
|||||||
}
|
}
|
||||||
)
|
)
|
||||||
|
|
||||||
return members_data
|
return sorted(members_data, key=lambda member: member["user_name"])
|
||||||
|
|
||||||
|
|
||||||
def get_new_member_form(application):
|
def get_new_member_form(application):
|
||||||
env_roles = [
|
env_roles = sorted(
|
||||||
|
[
|
||||||
{"environment_id": e.id, "environment_name": e.name}
|
{"environment_id": e.id, "environment_name": e.name}
|
||||||
for e in application.environments
|
for e in application.environments
|
||||||
]
|
],
|
||||||
|
key=lambda role: role["environment_name"],
|
||||||
|
)
|
||||||
|
|
||||||
return NewMemberForm(data={"environment_roles": env_roles})
|
return NewMemberForm(data={"environment_roles": env_roles})
|
||||||
|
|
||||||
|
@ -52,7 +52,10 @@ def serialize_member_form_data(member):
|
|||||||
|
|
||||||
|
|
||||||
def get_members_data(portfolio):
|
def get_members_data(portfolio):
|
||||||
members = [serialize_member_form_data(member) for member in portfolio.members]
|
members = sorted(
|
||||||
|
[serialize_member_form_data(member) for member in portfolio.members],
|
||||||
|
key=lambda member: member["member_name"],
|
||||||
|
)
|
||||||
for member in members:
|
for member in members:
|
||||||
if member["member_id"] == portfolio.owner_role.id:
|
if member["member_id"] == portfolio.owner_role.id:
|
||||||
ppoc = member
|
ppoc = member
|
||||||
|
Loading…
x
Reference in New Issue
Block a user