darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-05-29 18:40:10 -04:00
Code Issues Packages Projects Releases Wiki Activity

Remove job_status from allowed properties

Browse Source 
We can’t guarantee it’s always present, so shouldn’t allow direct access
to it.
This commit is contained in:
Chris Hill-Scott
2020-01-20 16:47:09 +00:00
parent 32105b3328
commit 0202f73f9a
2 changed files with 1 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
app/models/job.py
View File

@@ -24,7 +24,6 @@ class Job(JSONModel):
'original_file_name',
'created_at',
'notification_count',
'job_status',
'created_by',
}

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

@@ -47,7 +47,7 @@
notifications,
caption=uploaded_file_name,
caption_visible=False,
empty_message='These messages have been deleted because they were sent more than {} days ago'.format(service_data_retention_days) if job.job_status == 'finished' else 'No messages to show yet…',
empty_message='These messages have been deleted because they were sent more than {} days ago'.format(service_data_retention_days) if job.status == 'finished' else 'No messages to show yet…',
field_headings=[
'Recipient',
'Status'