darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-02-03 09:51:11 -05:00
Code Issues Packages Projects Releases Wiki Activity

ensure cronitor decorator is inside the notify_task wrapper

Browse Source 
the celery decorator should always be on the outside so that all other
decorators will be captured within the celery task. We had problems
with cronitor not reporting, and only for this task.
This commit is contained in:
Leo Hemsted
2019-06-03 11:46:07 +01:00
parent b0baf10229
commit 09888f7479
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/celery/letters_pdf_tasks.py
View File

@@ -111,8 +111,8 @@ def get_letters_pdf(template, contact_block, filename, values):
return resp.content, billable_units
@cronitor("collate-letter-pdfs-for-day")
@notify_celery.task(name='collate-letter-pdfs-for-day')
@cronitor("collate-letter-pdfs-for-day")
def collate_letter_pdfs_for_day(date=None):
if not date:
# Using the truncated date is ok because UTC to BST does not make a difference to the date,