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

Merge pull request #1167 from GSA/notify-admin-1148

Browse Source 
notify-admin-1148 make sure one-off files get written to db
This commit is contained in:
Carlo Costino
2024-01-30 11:38:58 -05:00
committed by GitHub
parent 4e488fa079 7c5674c4e1
commit f097b9ca07
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(