darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-07-10 19:34:27 -04:00
Code Issues Packages Projects Releases Wiki Activity

Map ‘pending’ status to ‘sending’

Browse Source 
For some reason we are getting pending back as a status from one of
our text message providers.

Because this is unrecognised by the admin app it shows up in the list of
messages as the raw status, styled like an error.

This commit maps it to sending instead, because the user doesn’t need to
know or care about the difference.
This commit is contained in:
Chris Hill-Scott
2018-09-11 13:46:33 +01:00
parent 916ea7c4fd
commit c59142b871
1 changed files with 6 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
app/__init__.py
View File

@@ -372,6 +372,7 @@ def format_notification_status(status, template_type):
'delivered': 'Delivered',
'sending': 'Sending',
'created': 'Sending',
'pending': 'Sending',
'sent': 'Sent internationally'
},
'letter': {
@@ -391,10 +392,9 @@ def format_notification_status(status, template_type):
def format_notification_status_as_time(status, created, updated):
return {
'sending': ' since {}'.format(created),
'created': ' since {}'.format(created)
}.get(status, updated)
return dict.fromkeys(
{'created', 'pending', 'sending'}, ' since {}'.format(created)
).get(status, updated)
def format_notification_status_as_field_status(status, notification_type):
@@ -423,7 +423,8 @@ def format_notification_status_as_field_status(status, notification_type):
'delivered': None,
'sent': None,
'sending': 'default',
'created': 'default'
'created': 'default',
'pending': 'default',
}
).get(status, 'error')