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

exclude CYSP from sanitise

Browse Source 
we know their content is already good, but they slightly exceed the
margins. Until they deploy their fixed template, lets not valiate them
This commit is contained in:
Leo Hemsted
2018-10-15 16:07:22 +01:00
parent e22e7245fe
commit 9606513e1f
1 changed files with 5 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
app/celery/letters_pdf_tasks.py
View File

@@ -187,6 +187,11 @@ def process_virus_scan_passed(self, filename):
new_pdf = _sanitise_precomiled_pdf(self, notification, old_pdf)
# TODO: Remove this once CYSP update their template to not cross over the margins
if notification.service_id == 'fe44178f-3b45-4625-9f85-2264a36dd9ec': # CYSP
# Check your state pension submit letters with good addresses and notify tags, so just use their supplied pdf
new_pdf = old_pdf
if not new_pdf:
current_app.logger.info('Invalid precompiled pdf received {} ({})'.format(notification.id, filename))
update_notification_status_by_id(notification.id, NOTIFICATION_VALIDATION_FAILED)