darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-08-11 09:28:27 -04:00
Code Issues Packages Projects Releases Wiki Activity

Remove confusing argument name

Browse Source 
Brings in new utils which changes the name of this argument to be closer
to what the API uses.
This commit is contained in:
Chris Hill-Scott
2017-11-16 13:35:17 +00:00
parent a358acfd05
commit 8d38215e86
3 changed files with 5 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app/__init__.py
View File

@@ -24,7 +24,7 @@ from flask_wtf.csrf import CSRFError
from functools import partial
from notifications_python_client.errors import HTTPError
from notifications_utils import logging, request_id, formatters
from notifications_utils import logging, request_helper, formatters
from notifications_utils.clients.statsd.statsd_client import StatsdClient
from notifications_utils.recipients import (
validate_phone_number,
@@ -93,7 +93,7 @@ def create_app(application):
statsd_client.init_app(application)
logging.init_app(application, statsd_client)
init_csrf(application)
request_id.init_app(application)
request_helper.init_app(application)
service_api_client.init_app(application)
user_api_client.init_app(application)

3
app/utils.py
View File

@@ -288,7 +288,8 @@ def get_template(
return SMSPreviewTemplate(
template,
prefix=service['name'],
sender=not service['prefix_sms'],
show_prefix=service['prefix_sms'],
sender=sms_sender,
show_recipient=show_recipient,
redact_missing_personalisation=redact_missing_personalisation,
)

2
requirements.txt
View File

@@ -21,4 +21,4 @@ notifications-python-client==4.6.0
awscli>=1.11,<1.12
awscli-cwlogs>=1.4,<1.5
git+https://github.com/alphagov/notifications-utils.git@21.5.0#egg=notifications-utils==21.5.0
git+https://github.com/alphagov/notifications-utils.git@23.0.0#egg=notifications-utils==23.0.0