From 4fc74984153254f995bdf0fcf4dc2028781c7dbc Mon Sep 17 00:00:00 2001 From: Chris Hill-Scott Date: Wed, 15 May 2019 10:04:43 +0100 Subject: [PATCH] Include status when linking to notifications page MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Without this argument the 'sending' filter doesn’t get highlighted by default. --- app/main/views/notifications.py | 1 + tests/app/main/views/test_notifications.py | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/app/main/views/notifications.py b/app/main/views/notifications.py index fb6143a5a..f4defa9b7 100644 --- a/app/main/views/notifications.py +++ b/app/main/views/notifications.py @@ -114,6 +114,7 @@ def view_notification(service_id, notification_id): 'main.view_notifications', service_id=current_service.id, message_type=template.template_type, + status='sending,delivered,failed', ) return render_template( diff --git a/tests/app/main/views/test_notifications.py b/tests/app/main/views/test_notifications.py index 8f9fd9e7f..228a662c7 100644 --- a/tests/app/main/views/test_notifications.py +++ b/tests/app/main/views/test_notifications.py @@ -118,7 +118,7 @@ def test_notification_status_page_respects_redaction( @pytest.mark.parametrize('extra_args, expected_back_link', [ ( {}, - partial(url_for, 'main.view_notifications', message_type='sms'), + partial(url_for, 'main.view_notifications', message_type='sms', status='sending,delivered,failed'), ), ( {'from_job': 'job_id'},