darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-09-06 15:08:25 -04:00
Code Issues Packages Projects Releases Wiki Activity

Include status when linking to notifications page

Browse Source 
Without this argument the 'sending' filter doesn’t get highlighted by
default.
This commit is contained in:
Chris Hill-Scott
2019-05-15 10:04:43 +01:00
parent b4894e7a03
commit 4fc7498415
2 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
app/main/views/notifications.py
View File

@@ -114,6 +114,7 @@ def view_notification(service_id, notification_id):
'main.view_notifications', 'main.view_notifications',
service_id=current_service.id, service_id=current_service.id,
message_type=template.template_type, message_type=template.template_type,
status='sending,delivered,failed',
) )
return render_template( return render_template(

2
tests/app/main/views/test_notifications.py
View File

@@ -118,7 +118,7 @@ def test_notification_status_page_respects_redaction(
@pytest.mark.parametrize('extra_args, expected_back_link', [ @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'}, {'from_job': 'job_id'},