darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2025-12-23 17:01:35 -05:00
Code Issues Packages Projects Releases Wiki Activity

Stop passing dvla_org_id to template preview

Browse Source 
We were passing both dvla_org_id and filename to template-preview
temporarily while we switch to only using filename. Now that
template-preview is set up to use the filename, we can stop sending the
dvla_org_id too.
This commit is contained in:
Katie Smith
2018-10-23 15:52:44 +01:00
parent 08caaec0f8
commit 022b5b19ff
3 changed files with 2 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app/celery/letters_pdf_tasks.py
View File

@@ -53,7 +53,6 @@ def create_letters_pdf(self, notification_id):
pdf_data, billable_units = get_letters_pdf(
notification.template,
contact_block=notification.reply_to_text,
org_id=notification.service.dvla_organisation.id,
filename=notification.service.dvla_organisation.filename,
values=notification.personalisation
)
@@ -80,7 +79,7 @@ def create_letters_pdf(self, notification_id):
update_notification_status_by_id(notification_id, 'technical-failure')
def get_letters_pdf(template, contact_block, org_id, filename, values):
def get_letters_pdf(template, contact_block, filename, values):
template_for_letter_print = {
"subject": template.subject,
"content": template.content
@@ -91,7 +90,6 @@ def get_letters_pdf(template, contact_block, org_id, filename, values):
'template': template_for_letter_print,
'values': values,
'filename': filename,
'dvla_org_id': org_id,
}
resp = requests_post(
'{}/print.pdf'.format(