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

Fix 500 on job page when notifications are sending

Browse Source 
Sending notifications don’t have an `updated_at`. This causes the time
formatting to throw a wobly, because it doesn’t expect `None`.

This commit changes the template to also look for the `created_at`,
which all notifications have.
This commit is contained in:
Chris Hill-Scott
2016-06-06 13:30:39 +01:00
parent 3ddc1d171d
commit 5b5ad2c5dc
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/templates/partials/jobs/notifications.html
View File

@@ -28,7 +28,7 @@
align='right',
status='error' if item.status == 'Failed' else 'default'
) %}
{{ item.status|format_notification_status }} at {{ item.updated_at|format_time }}
{{ item.status|format_notification_status }} at {{ (item.updated_at or item.created_at)|format_time }}
{% endcall %}
{% endcall %}
{% else %}

2
app/templates/views/notifications.html
View File

@@ -94,7 +94,7 @@
{{ text_field(item.status|format_notification_status) }}
{% call field(align='right') %}
{{ item.updated_at|format_datetime_short }}
{{ (item.updated_at or item.created_at)|format_datetime_short }}
{% endcall %}
{% endcall %}