darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-01-03 14:30:48 -05:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #3268 from alphagov/update-job-sooner

Browse Source 
Update the job_status to in-progress sooner.
This commit is contained in:
Rebecca Law
2021-06-15 15:45:52 +01:00
committed by GitHub
parent 92d52de68f 8af10eb1f0
commit 6d09313b18
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
app/celery/tasks.py
View File

@@ -71,6 +71,10 @@ def process_job(job_id, sender_id=None):
service = job.service
job.job_status = JOB_STATUS_IN_PROGRESS
job.processing_started = start
dao_update_job(job)
if not service.active:
job.job_status = JOB_STATUS_CANCELLED
dao_update_job(job)
@@ -81,10 +85,6 @@ def process_job(job_id, sender_id=None):
if __sending_limits_for_job_exceeded(service, job, job_id):
return
job.job_status = JOB_STATUS_IN_PROGRESS
job.processing_started = start
dao_update_job(job)
recipient_csv, template, sender_id = get_recipient_csv_and_template_and_sender_id(job)
current_app.logger.info("Starting job {} processing {} notifications".format(job_id, job.notification_count))