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

Merge pull request #1964 from GSA/notify-admin-1956

Browse Source 
make sure phone number is always last in the one-off csv
This commit is contained in:
Carlo Costino
2024-09-30 13:03:09 -04:00
committed by GitHub
parent 49ec3dc559 bdc5614fff
commit 2c7d090b82
1 changed files with 9 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
app/main/views/send.py
View File

@@ -1009,9 +1009,16 @@ def _send_notification(service_id, template_id):
keys = [] keys = []
values = [] values = []
# Guarantee that the real phone number comes last, because some
# users will have placeholders like "add your second phone number"
# or something like as custom placeholders.
for k, v in session["placeholders"].items(): for k, v in session["placeholders"].items():
keys.append(k) if k != "phone number":
values.append(v) keys.append(k)
values.append(v)
if "phone number" in session["placeholders"].keys():
keys.append("phone number")
values.append(session["placeholders"]["phone number"])
data = ",".join(keys) data = ",".join(keys)
vals = ",".join(values) vals = ",".join(values)