diff --git a/src/registrar/admin.py b/src/registrar/admin.py index 699912f8e..1b1e0978f 100644 --- a/src/registrar/admin.py +++ b/src/registrar/admin.py @@ -1816,9 +1816,9 @@ class PortfolioInvitationAdmin(BaseInvitationAdmin): def get_roles(self, obj): readable_roles = obj.get_readable_roles() return ", ".join(readable_roles) - + get_roles.short_description = "Member access" # type: ignore - + # Select portfolio invitations to change -> Portfolio invitations def changelist_view(self, request, extra_context=None): if extra_context is None: diff --git a/src/registrar/models/portfolio_invitation.py b/src/registrar/models/portfolio_invitation.py index 5c5c62142..1da0fcdd1 100644 --- a/src/registrar/models/portfolio_invitation.py +++ b/src/registrar/models/portfolio_invitation.py @@ -82,7 +82,7 @@ class PortfolioInvitation(TimeStampedModel): def get_readable_roles(self): """Returns a readable list of self.roles""" return get_readable_roles(self.roles) - + def get_managed_domains_count(self): """Return the count of domain invitations managed by the invited user for this portfolio.""" # Filter the UserDomainRole model to get domains where the user has a manager role diff --git a/src/registrar/models/utility/portfolio_helper.py b/src/registrar/models/utility/portfolio_helper.py index 70be16d5b..03dca897f 100644 --- a/src/registrar/models/utility/portfolio_helper.py +++ b/src/registrar/models/utility/portfolio_helper.py @@ -82,11 +82,10 @@ class MemberPermissionDisplay(StrEnum): def get_readable_roles(roles): readable_roles = [] if roles: - readable_roles = sorted( - [UserPortfolioRoleChoices.get_user_portfolio_role_label(role) for role in roles] - ) + readable_roles = sorted([UserPortfolioRoleChoices.get_user_portfolio_role_label(role) for role in roles]) return readable_roles + def get_role_display(roles): """ Returns a user-friendly display name for a given list of user roles.