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

Make ‘temporary failure’ error less cryptic

Browse Source 
‘Phone number doesn’t exist’ as opposed to ‘Permanent failure’ has
tested well because it talks in terms of things people understand.

We should do the same for ‘Temporary failure’, because users are
unclear:
- why this is happening
- if it’s temporary, is Notify going to retry it (it’s not)

We think that ‘Phone/inbox not currently accepting messages’ answers
makes these things clearer.

I’ve reworded it slightly to:

- ‘Inbox not accepting messages right now’
- ‘Phone not accepting messages right now’
This commit is contained in:
Chris Hill-Scott
2016-07-11 10:45:54 +01:00
parent d33c254fa5
commit f9a9048579
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
app/__init__.py
View File

@@ -235,16 +235,16 @@ def format_notification_status(status, template_type):
'email': { 'email': {
'failed': 'Failed', 'failed': 'Failed',
'technical-failure': 'Technical failure', 'technical-failure': 'Technical failure',
'temporary-failure': 'Temporary failure', 'temporary-failure': 'Inbox not accepting messages right now',
'permanent-failure': 'Email address does not exist', 'permanent-failure': 'Email address doesn’t exist',
'delivered': 'Delivered', 'delivered': 'Delivered',
'sending': 'Sending' 'sending': 'Sending'
}, },
'sms': { 'sms': {
'failed': 'Failed', 'failed': 'Failed',
'technical-failure': 'Technical failure', 'technical-failure': 'Technical failure',
'temporary-failure': 'Temporary failure', 'temporary-failure': 'Phone not accepting messages right now',
'permanent-failure': 'Phone number does not exist', 'permanent-failure': 'Phone number doesn’t exist',
'delivered': 'Delivered', 'delivered': 'Delivered',
'sending': 'Sending' 'sending': 'Sending'
} }