darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-02-04 18:31:13 -05:00
Code Issues Packages Projects Releases Wiki Activity

make sure log line works if notification still in created

Browse Source
This commit is contained in:
Leo Hemsted
2018-12-28 14:29:59 +00:00
parent 2355ee011f
commit 021625abb3
1 changed files with 2 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
app/dao/notifications_dao.py
View File

@@ -185,8 +185,7 @@ def update_notification_status_by_reference(reference, status):
if notification.status not in { if notification.status not in {
NOTIFICATION_SENDING, NOTIFICATION_SENDING,
NOTIFICATION_PENDING, NOTIFICATION_PENDING
NOTIFICATION_SENT,
}: }:
_duplicate_update_warning(notification, status) _duplicate_update_warning(notification, status)
return None return None
@@ -714,7 +713,7 @@ def _duplicate_update_warning(notification, status):
id=notification.id, id=notification.id,
old_status=notification.status, old_status=notification.status,
new_status=status, new_status=status,
time_diff=datetime.utcnow() - notification.sent_at, time_diff=datetime.utcnow() - (notification.updated_at or notification.created_at),
type=notification.notification_type, type=notification.notification_type,
sent_by=notification.sent_by sent_by=notification.sent_by
) )