Merge pull request #1140 from alphagov/2127-detailed-reminders-email-info

2127: Add more information into detailed case reminder emails

Author: ahernp

accessibility_monitoring_platform/apps/notifications/templates/notifications/all_detailed_reminders_email.txt

@@ -1,20 +1,20 @@
 Here are all of the reminders for detailed Cases which are overdue or due in the next seven days.
 
 {% for task in tasks %}
-
-User: {% if task.user %}{{ task.user.get_full_name }}{% elif task.base_case.auditor %}{{ task.base_case.auditor.get_full_name }}{% else %}Unknown{% endif %}
+User who set reminder: {{ task.user.get_full_name }}
 
 Date: {{ task.date|amp_date }}
 
 ID: {{ task.base_case.case_identifier }}
 
-Case: {{ task.base_case.organisation_name }}
+Case: {% if task.base_case.website_name %}{{ task.base_case.website_name }} ({{ task.base_case.organisation_name }}){% else %}{{ task.base_case.organisation_name }}{% endif %}
 See https://platform.accessibility-monitoring.service.gov.uk{{ task.base_case.get_absolute_url }}
 
-Description
+Auditor: {% if task.base_case.auditor %}{{ task.base_case.auditor.get_full_name }}{% else %}None{% endif %}
+
+Description:
 
 {{ task.description }}
 
 ------
-
 {% endfor %}

accessibility_monitoring_platform/apps/notifications/tests/test_utils.py

@@ -1046,7 +1046,9 @@ def test_email_all_specialists_all_detailed_reminders_due(mailoutbox):
     specialist_group.user_set.add(specialist_2)
 
     detailed_case_1: DetailedCase = DetailedCase.objects.create(
-        organisation_name="Detailed Organisation One"
+        organisation_name="Detailed Organisation One",
+        website_name="Website name",
+        auditor=specialist_1,
     )
     Task.objects.create(
         type=Task.Type.REMINDER,
@@ -1090,3 +1092,5 @@ def test_email_all_specialists_all_detailed_reminders_due(mailoutbox):
     assert "First reminder description" in mailoutbox[0].body
     assert "Second reminder description" in mailoutbox[0].body
     assert "Third reminder description" not in mailoutbox[0].body
+
+    # print(mailoutbox[0].message())  # Visible when run with pytest -s