Merge pull request #1931 from alphagov/letter-status-colour

Don’t change colour of sent date for letters
This commit is contained in:
Chris Hill-Scott
2018-03-09 09:51:40 +00:00
committed by GitHub
3 changed files with 30 additions and 12 deletions

View File

@@ -356,17 +356,32 @@ def format_notification_status_as_time(status, created, updated):
}.get(status, updated)
def format_notification_status_as_field_status(status):
def format_notification_status_as_field_status(status, notification_type):
return {
'failed': 'error',
'technical-failure': 'error',
'temporary-failure': 'error',
'permanent-failure': 'error',
'delivered': None,
'sent': None,
'sending': 'default',
'created': 'default'
}.get(status, 'error')
'letter': {
'failed': 'error',
'technical-failure': 'error',
'temporary-failure': 'error',
'permanent-failure': 'error',
'delivered': None,
'sent': None,
'sending': None,
'created': None,
'accepted': None,
}
}.get(
notification_type,
{
'failed': 'error',
'technical-failure': 'error',
'temporary-failure': 'error',
'permanent-failure': 'error',
'delivered': None,
'sent': None,
'sending': 'default',
'created': 'default'
}
).get(status, 'error')
def format_notification_status_as_url(status):

View File

@@ -130,7 +130,10 @@
{% if not notification %}
{% call field(align='right') %}{% endcall %}
{% else %}
{% call field(status=notification.status|format_notification_status_as_field_status, align='right') %}
{% call field(
status=notification.status|format_notification_status_as_field_status(notification.notification_type),
align='right'
) %}
{% if notification.status in ['created', 'sending', 'delivered'] %}<span class="align-with-message-body">{% endif %}
{% if notification.status|format_notification_status_as_url %}
<a href="{{ notification.status|format_notification_status_as_url }}">

View File

@@ -1,5 +1,5 @@
<div class="ajax-block-container">
<p class="notification-status {{ notification.status|format_notification_status_as_field_status }}">
<p class="notification-status {{ notification.status|format_notification_status_as_field_status(notification.notification_type) }}">
{% if notification.status|format_notification_status_as_url %}
<a href="{{ notification.status|format_notification_status_as_url }}">
{% endif %}