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

notify-admin-1137 make sure one-off files get written to db

Browse Source
This commit is contained in:
Kenneth Kehl
2024-01-29 13:17:50 -08:00
parent 4e488fa079
commit 7c5674c4e1
1 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
app/main/views/send.py
View File

@@ -880,14 +880,20 @@ def send_notification(service_id, template_id):
service_id, job_id=upload_id, include_one_off=True service_id, job_id=upload_id, include_one_off=True
) )
attempts = 0 attempts = 0
while notifications["total"] == 0 and attempts < 5:
# The response can come back in different forms of incompleteness
while (
notifications["total"] == 0
and notifications["notifications"] == []
and attempts < 50
):
notifications = notification_api_client.get_notifications_for_service( notifications = notification_api_client.get_notifications_for_service(
service_id, job_id=upload_id, include_one_off=True service_id, job_id=upload_id, include_one_off=True
) )
time.sleep(0.1) time.sleep(0.1)
attempts = attempts + 1 attempts = attempts + 1
if notifications["total"] == 0 and attempts == 5: if notifications["total"] == 0 and attempts == 50:
# This shows the job we auto-generated for the user # This shows the job we auto-generated for the user
return redirect( return redirect(
url_for( url_for(