Skip to content

Commit 41d960e

Browse files
authored
Merge pull request #1121 from alphagov/2112-7b-additional-issues
2112: Include additional WCAG issues in email 7b even if page has no outstanding original issues
2 parents ac5947f + 719d630 commit 41d960e

1 file changed

Lines changed: 12 additions & 12 deletions

File tree

accessibility_monitoring_platform/apps/common/templates/common/emails/templates/7b-equality-body-retest-unfixed-issues.html

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ <h2 class="amp-margin-bottom-0">Correspondence</h2>
102102
<h2 class="amp-margin-bottom-0">Errors found in retest</h2>
103103
<br>
104104
{% for retest_page in retest.retestpage_set.all %}
105-
{% if retest_page.missing_date or retest_page.unfixed_check_results %}
105+
{% if retest_page.missing_date or retest_page.unfixed_check_results or retest_page.additional_issues_notes %}
106106
<b>{{ retest_page.page }}{% if retest_page.page.page_type != 'pdf' %} page{% endif %} issues</b>
107107
({{ retest_page.unfixed_check_results.count }} issue{% if retest_page.unfixed_check_results.count != 1 %}s{% endif %} remaining)
108108
<br>
@@ -113,7 +113,7 @@ <h2 class="amp-margin-bottom-0">Errors found in retest</h2>
113113
This page has been removed by the organisation.
114114
<br>
115115
<br>
116-
{% else %}
116+
{% elif retest_page.unfixed_check_results %}
117117
<table id="email-issues-table-{{ forloop.counter }}">
118118
<thead>
119119
<tr>
@@ -158,16 +158,16 @@ <h2 class="amp-margin-bottom-0">Errors found in retest</h2>
158158
{% endfor %}
159159
</tbody>
160160
</table>
161-
<b>Additional issues found</b>
162-
{% if retest_page.additional_issues_notes %}
163-
{{ retest_page.additional_issues_notes|markdown_to_html }}
164-
{% else %}
165-
<br>
166-
<br>
167-
None
168-
<br>
169-
<br>
170-
{% endif %}
161+
{% endif %}
162+
<b>Additional issues found</b>
163+
{% if retest_page.additional_issues_notes %}
164+
{{ retest_page.additional_issues_notes|markdown_to_html }}
165+
{% else %}
166+
<br>
167+
<br>
168+
None
169+
<br>
170+
<br>
171171
{% endif %}
172172
{% endif %}
173173
{% endfor %}

0 commit comments

Comments
 (0)