darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2025-12-21 07:51:13 -05:00
Code Issues Packages Projects Releases Wiki Activity

Update template content values from notification personalisation

Browse Source 
We need to set template_with_content placeholder values with an
updated personalisation dictionary after processing the document
uploads in order to avoid returning base64 file data instead of
the document URLs in the response contents.
This commit is contained in:
Alexey Bezhan
2018-04-09 16:27:56 +01:00
parent b097a16a86
commit c189f09687
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
app/v2/notifications/post_notifications.py
View File

@@ -138,7 +138,7 @@ def post_notification(notification_type):
reply_to_text=reply_to reply_to_text=reply_to
) )
else: else:
notification, personalisation = process_sms_or_email_notification( notification = process_sms_or_email_notification(
form=form, form=form,
notification_type=notification_type, notification_type=notification_type,
api_key=api_user, api_key=api_user,
@@ -147,7 +147,7 @@ def post_notification(notification_type):
reply_to_text=reply_to reply_to_text=reply_to
) )
template_with_content.values = personalisation template_with_content.values = notification.personalisation
if notification_type == SMS_TYPE: if notification_type == SMS_TYPE:
create_resp_partial = functools.partial( create_resp_partial = functools.partial(
@@ -216,7 +216,7 @@ def process_sms_or_email_notification(*, form, notification_type, api_key, templ
else: else:
current_app.logger.debug("POST simulated notification for id: {}".format(notification.id)) current_app.logger.debug("POST simulated notification for id: {}".format(notification.id))
return notification, personalisation return notification
def process_document_uploads(personalisation_data, service, simulated=False): def process_document_uploads(personalisation_data, service, simulated=False):