Merge pull request #1133 from alphagov/2121-show-all-detailed-reminders

2121: Add option to show all undeleted detailed reminders on Task list

Author: ahernp

accessibility_monitoring_platform/apps/notifications/templates/notifications/task_list.html

@@ -132,6 +132,13 @@ <h1 class="govuk-heading-xl amp-margin-bottom-30">{{ sitemap.current_platform_pa
                                 <a href="?type=reminder&future=true" class="govuk-link govuk-link--no-visited-state govuk-link--no-underline">View future</a>
                             {% endif %}
                         </li>
+                        <li class="amp-custom-nav-bar amp-margin-right-25">
+                            {% if show_all_detailed_reminders %}
+                                <b> All detailed </b>
+                            {% else %}
+                                <a href="?type=reminder&show_all_detailed_reminders=true" class="govuk-link govuk-link--no-visited-state govuk-link--no-underline">All detailed</a>
+                            {% endif %}
+                        </li>
                     </ul>
                 {% endif %}
             </div>

accessibility_monitoring_platform/apps/notifications/tests/test_views.py

@@ -698,3 +698,51 @@ def test_deactivate_case_updates_status(admin_client):
     )
 
     assert simplified_case_from_db.status == SimplifiedCase.Status.DEACTIVATED
+
+
+@pytest.mark.django_db
+def test_task_list_show_all_detailed_reminders(rf):
+    """Test task list page can show all due reminder tasks for detailed cases"""
+    request_user: User = User.objects.create(
+        username="mockuser1", email="mockuser1@mock.com", password="secret1"
+    )
+    other_user: User = User.objects.create(
+        username="mockuser2", email="mockuser2@mock.com", password="secret2"
+    )
+    simplified_case: SimplifiedCase = SimplifiedCase.objects.create(auditor=other_user)
+    Task.objects.create(
+        type=Task.Type.REMINDER,
+        date=date.today(),
+        user=request_user,
+        base_case=simplified_case,
+        description="Simplified reminder description",
+    )
+    detailed_case: DetailedCase = DetailedCase.objects.create(auditor=other_user)
+    Task.objects.create(
+        type=Task.Type.REMINDER,
+        date=date.today(),
+        user=request_user,
+        base_case=detailed_case,
+        description="First detailed reminder description",
+    )
+    Task.objects.create(
+        type=Task.Type.REMINDER,
+        date=date.today(),
+        user=other_user,
+        base_case=detailed_case,
+        description="Second detailed reminder description",
+    )
+
+    request: HttpRequest = rf.get(
+        f'{reverse("notifications:task-list")}?type=reminder&show_all_detailed_reminders=true'
+    )
+    request.user = request_user
+
+    response: HttpResponse = TaskListView.as_view()(request)
+
+    assert response.status_code == 200
+
+    assertContains(response, "Tasks (2)")
+    assertContains(response, "First detailed reminder description")
+    assertContains(response, "Second detailed reminder description")
+    assertNotContains(response, "Simplified reminder description")

accessibility_monitoring_platform/apps/notifications/views.py

@@ -4,6 +4,7 @@
 
 from django.contrib import messages
 from django.contrib.auth.models import User
+from django.db.models.query import QuerySet
 from django.forms.models import ModelForm
 from django.http import HttpResponseRedirect
 from django.urls import reverse, reverse_lazy
@@ -44,6 +45,17 @@ def get_context_data(self, **kwargs) -> dict[str, Any]:
             context["task_type_counts"] = get_task_type_counts(tasks=tasks)
             return {**context, **params}
 
+        if "show_all_detailed_reminders" in self.request.GET:
+            tasks: QuerySet[Task] = Task.objects.filter(
+                type=Task.Type.REMINDER,
+                base_case__test_type=BaseCase.TestType.DETAILED,
+                read=False,
+            ).order_by("date")
+            context["tasks"] = tasks
+            context["show_all_detailed_reminders"] = True
+            context["task_type_counts"] = get_task_type_counts(tasks=tasks)
+            return {**context, **params}
+
         user: User = self.request.user
 
         # Check for parameter to list another user's Tasks. Useful for live support.