diff --git a/src/registrar/templates/domain_request_review.html b/src/registrar/templates/domain_request_review.html index aeba9955c..09a930710 100644 --- a/src/registrar/templates/domain_request_review.html +++ b/src/registrar/templates/domain_request_review.html @@ -156,10 +156,7 @@ {% if step == Step.ADDITIONAL_DETAILS %} - {% namespaced_url 'domain-request' step as domain_request_url %} - {% with title=form_titles|get_item:step value=domain_request.anything_else|default:"No" %} - {% include "includes/summary_item.html" with title=title value=value heading_level=heading_level editable=True edit_link=domain_request_url %} - {% endwith %} + {% include "includes/summary_additional_details.html" with domainRequest=DomainRequest %} {% endif %} diff --git a/src/registrar/templates/domain_request_status.html b/src/registrar/templates/domain_request_status.html index d3c1eab6d..cf9e7ffe3 100644 --- a/src/registrar/templates/domain_request_status.html +++ b/src/registrar/templates/domain_request_status.html @@ -115,8 +115,8 @@ {% else %} {% include "includes/summary_item.html" with title='Other employees from your organization' value=DomainRequest.no_other_contacts_rationale heading_level=heading_level %} {% endif %} - - {% include "includes/summary_item.html" with title='Anything else?' value=DomainRequest.anything_else|default:"No" heading_level=heading_level %} + + {% include "includes/summary_additional_details.html" with domainRequest=DomainRequest %} {% endwith %} diff --git a/src/registrar/templates/includes/summary_additional_details.html b/src/registrar/templates/includes/summary_additional_details.html new file mode 100644 index 000000000..e10b90384 --- /dev/null +++ b/src/registrar/templates/includes/summary_additional_details.html @@ -0,0 +1,50 @@ +{% load static url_helpers %} + +{# TODO (future ticket?): create a template that allows us to easily display nested forms. +It is a little complex because we will have to formulate how to aggregate form pairings. +(eg. yes/no radio forms plus the information they toggle need to be linked somehow +and condense down into one subsection)} + +
+ +
+
+

+ Additional Details +

+ + {% if domainRequest %} +
+
+ CISA regions representative +
+
+ {% if domainRequest.has_cisa_representative %} + domainRequest.cisa_representative_email + {% else %} + (none) + {% endif %} +
+
+ Anything else +
+
+ {% if domainRequest.has_anything_else_text %} + domainRequest.anything_else + {% else %} + No + {% endif %} +
+
+ {% else %} + ERROR Please contact technical support/dev + {% endif %} +
+
+