zydronium/manage.get.gov
1
0
Fork 0
mirror of https://github.com/cisagov/manage.get.gov.git synced 2025-08-02 16:02:15 +02:00
Code Issues Projects Releases Packages Wiki Activity

#3866 - Add Organization admins to the Organization overview page - [meoward] (#3928)

Browse source 
Added logic to existing summary_item.html
Added statement to pull portfolio admins in view, portfolios.py, for summary_item.html display
Changed porfolio_suborganization.html to reflect requirements of ticket
This commit is contained in:
Abe Alam 2025-07-18 10:10:46 -04:00 • committed by GitHub
parent 033d628eac
commit aed8b9b2b7
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
4 changed files with 132 additions and 104 deletions
Download patch file Download diff file Expand all files Collapse all files

222
src/registrar/templates/includes/summary_item.html
View file

@ -1,7 +1,9 @@
{% load static url_helpers %}
{% if portfolio_first_section %} <section class="summary-item margin-top-2">
{% else %} <section class="summary-item margin-top-3">
{% if portfolio_first_section %}
<section class="summary-item margin-top-2">
{% else %}
<section class="summary-item margin-top-3">
{% endif %}
<hr class="" aria-hidden="true" />
<div class="display-flex flex-justify">
@ -20,120 +22,137 @@
{% else %}
</h3>
{% endif %}
{% if sub_header_text %}
<h4 class="margin-bottom-0">{{ sub_header_text }}</h4>
{# To display default styling for sub-header text and organization information in organization overview #}
{% if sub_header_text %}
<h4 class="margin-bottom-0">{{ sub_header_text }}</h4>
{% if sub_header_text == "Organization Admins" %}
<ul class="usa-list">
{% for admin in portfolio_admins %}
<li>{{ admin.get_full_name|default:admin.username }}</li>
{% empty %}
<li>No admins assigned.</li>
{% endfor %}
</ul>
<h4 class="margin-bottom-2">Organization Name and Address</h4>
{% endif %}
{% if permissions %}
{% include "includes/member_permissions_summary.html" with permissions=value %}
{% elif domain_mgmt %}
{% include "includes/member_domain_management.html" with domain_count=value %}
{% elif address %}
{% include "includes/organization_address.html" with organization=value %}
{% elif contact %}
{% if list %}
{% if value|length == 1 %}
{% include "includes/contact.html" with contact=value|first %}
{% endif %}
{% if permissions %}
{% include "includes/member_permissions_summary.html" with permissions=value %}
{% elif domain_mgmt %}
{% include "includes/member_domain_management.html" with domain_count=value %}
{% elif address %}
{% include "includes/organization_address.html" with organization=value %}
{% elif contact %}
{% if list %}
{% if value|length == 1 %}
{% include "includes/contact.html" with contact=value|first %}
{% else %}
{% if value %}
<dl class="usa-list usa-list--unstyled margin-top-0">
{% for item in value %}
<dt>
<h4 class="margin-bottom-05 padding-right-1">
Contact {{forloop.counter}}
</h4>
</dt>
<dd>
{% include "includes/contact.html" with contact=item %}
</dd>
{% endfor %}
</dl>
{% elif custom_text_for_value_none %}
<p>
{{ custom_text_for_value_none }}
</p>
{% else %}
{% if value %}
<dl class="usa-list usa-list--unstyled margin-top-0">
{% for item in value %}
<dt>
<h4 class="
margin-bottom-05
padding-right-1">
Contact {{forloop.counter}}
</h4>
</dt>
<dd>
{% include "includes/contact.html" with contact=item %}
</dd>
{% endfor %}
</dl>
{% elif custom_text_for_value_none %}
<p>
{{ custom_text_for_value_none }}
</p>
{% else %}
<p>
None
</p>
{% endif %}
<p>
None
</p>
{% endif %}
{% endif %}
{% else %}
{% include "includes/contact.html" with contact=value %}
{% endif %}
{% elif list %}
{% if value|length == 1 %}
{% if users %}
<p class="margin-top-0">{{ value.0.user.email }} </p>
{% elif domains %}
{{ value.0.0 }}
{% if value.0.1 %}
({% spaceless %}
{% for addr in value.0.1 %}
{{addr}}{% if not forloop.last %}, {% endif %}
{% endfor %}
{% endspaceless %})
{% endif %}
{% else %}
{% include "includes/contact.html" with contact=value %}
<p class="margin-top-0 margin-bottom-0">{{ value | first }} </p>
{% endif %}
{% elif list %}
{% if value|length == 1 %}
{% if users %}
<p class="margin-top-0">{{ value.0.user.email }} </p>
{% elif domains %}
{{ value.0.0 }}
{% if value.0.1 %}
{% else %}
<ul class="usa-list usa-list--unstyled margin-top-0">
{% for item in value %}
{% if users %}
<li>{{ item.user.email }}</li>
{% elif domains %}
<li>
{{ item.0 }}
{% if item.1 %}
({% spaceless %}
{% for addr in value.0.1 %}
{% for addr in item.1 %}
{{addr}}{% if not forloop.last %}, {% endif %}
{% endfor %}
{% endspaceless %})
{% endif %}
{% else %}
<p class="margin-top-0 margin-bottom-0">{{ value | first }} </p>
{% endif %}
</li>
{% else %}
<li>{{ item }}</li>
{% endif %}
{% empty %}
<li>None</li>
{% endfor %}
</ul>
{% endif %}
{% elif domain_permissions %}
{% if value.permissions.all %}
{% if value.permissions|length == 1 %}
<p class="margin-top-0">{{ value.permissions.0.user.email }} </p>
{% else %}
<ul class="usa-list usa-list--unstyled margin-top-0">
{% for item in value %}
{% if users %}
<li>{{ item.user.email }}</li>
{% elif domains %}
<li>
{{ item.0 }}
{% if item.1 %}
({% spaceless %}
{% for addr in item.1 %}
{{addr}}{% if not forloop.last %}, {% endif %}
{% endfor %}
{% endspaceless %})
{% endif %}
</li>
{% else %}
<li>{{ item }}</li>
{% endif %}
{% empty %}
<li>None</li>
{% endfor %}
</ul>
{% endif %}
{% elif domain_permissions %}
{% if value.permissions.all %}
{% if value.permissions|length == 1 %}
<p class="margin-top-0">{{ value.permissions.0.user.email }} </p>
{% else %}
<ul class="usa-list usa-list--unstyled margin-top-0">
{% for item in value.permissions.all %}
<li>{{ item.user.email }}</li>
{% endfor %}
</ul>
{% endif %}
{% endif %}
{% if value.active_invitations.all %}
<h4 class="margin-bottom-05">Invited domain managers</h4>
<ul class="usa-list usa-list--unstyled margin-top-0">
{% for item in value.active_invitations.all %}
<li>{{ item.email }}</li>
{% for item in value.permissions.all %}
<li>{{ item.user.email }}</li>
{% endfor %}
</ul>
{% endif %}
{% else %}
{% if custom_text_for_value_none %}
<p class="margin-top-0 text-base-dark">{{ custom_text_for_value_none }}</p>
{% endif %}
{% if value %}
{{ value }}
{% endif %}
{% if not value %}
None
{% endif %}
{% endif %}
{% if value.active_invitations.all %}
<h4 class="margin-bottom-05">Invited domain managers</h4>
<ul class="usa-list usa-list--unstyled margin-top-0">
{% for item in value.active_invitations.all %}
<li>{{ item.email }}</li>
{% endfor %}
</ul>
{% endif %}
{% else %}
{% if custom_text_for_value_none %}
<p class="margin-top-0 text-base-dark">{{ custom_text_for_value_none }}</p>
{% endif %}
{% if value %}
{{ value }}
{% endif %}
{% if not value %}
None
{% endif %}
{% endif %}
</div>
{% comment %}We have conditions where an edit_link is set but editable can be true or false{% endcomment %}
@ -159,6 +178,5 @@
</div>
{% endif %}
{% endif %}
</div>
</div>
</section>

4
src/registrar/templates/portfolio_organization.html
View file

@ -20,8 +20,8 @@
<div class="margin-top-2 tablet:grid-col-10">
<h2 class="string-wrap margin-top-2">{{ portfolio }}</h2>
{% url 'organization-info' as url %}
{% include "includes/summary_item.html" with title='Organization' value=portfolio address='true' edit_link=url editable=has_edit_portfolio_permission view_button='true' portfolio_first_section='true'%}
{% include "includes/summary_item.html" with title='Organization' value=portfolio address='true' sub_header_text='Organization Admins' edit_link=url editable=has_edit_portfolio_permission view_button='true' portfolio_first_section='true' %}
{% url 'organization-senior-official' as url %}
{% include "includes/summary_item.html" with title='Senior official' value=portfolio.senior_official contact='true' edit_link=url view_button='true' %}
</div>

9
src/registrar/tests/test_views_portfolio.py
View file

@ -212,6 +212,13 @@ class TestPortfolio(WebTest):
first_name="Saturn", last_name="Enceladus", title="Planet/Moon", email="spacedivision@igorville.com"
)
portfolio_admin = User.objects.create_user(
username="adminuser", first_name="Galileo", last_name="Galilei", email="admin@example.com"
)
UserPortfolioPermission.objects.create(
user=portfolio_admin, portfolio=self.portfolio, roles=[UserPortfolioRoleChoices.ORGANIZATION_ADMIN]
)
self.portfolio.senior_official = so
self.portfolio.organization_name = "Hotel California"
self.portfolio.city = "Los Angeles"
@ -227,6 +234,8 @@ class TestPortfolio(WebTest):
# Organization overview page includes organization info and senior official details
self.assertContains(response, "Los Angeles")
self.assertContains(response, "spacedivision@igorville.com")
# Organization overview page includes portfolio admin
self.assertContains(response, "Galileo")
@less_console_noise_decorator
def test_portfolio_organization_page_directs_to_org_detail_forms(self):

1
src/registrar/views/portfolios.py
View file

@ -904,6 +904,7 @@ class PortfolioOrganizationView(DetailView):
context = super().get_context_data(**kwargs)
portfolio = self.request.session.get("portfolio")
context["has_edit_portfolio_permission"] = self.request.user.has_edit_portfolio_permission(portfolio)
context["portfolio_admins"] = portfolio.portfolio_admin_users
return context
def get_object(self, queryset=None):