diff --git a/app/assets/sass/uswds/_legacy-styles.scss b/app/assets/sass/uswds/_legacy-styles.scss index 48cd63df8..b96a24844 100644 --- a/app/assets/sass/uswds/_legacy-styles.scss +++ b/app/assets/sass/uswds/_legacy-styles.scss @@ -48,8 +48,8 @@ } } -.sms-message-sender, .sms-message-file-name, .sms-message-scheduler, .sms-message-template { - margin: units(1) 0 units(1); +.sms-message-sender, .sms-message-file-name, .sms-message-scheduler, .sms-message-template, .sms-message-sender { + margin:0.25rem 0 0; } .sms-message-recipient { diff --git a/app/main/views/send.py b/app/main/views/send.py index df53ef3c6..a8367f7da 100644 --- a/app/main/views/send.py +++ b/app/main/views/send.py @@ -39,7 +39,7 @@ from app.utils import PermanentRedirect, should_skip_template_page, unicode_trun from app.utils.csv import Spreadsheet, get_errors_for_csv from app.utils.templates import get_template from app.utils.user import user_has_permissions - +from pprint import pprint def get_example_csv_fields(column_headers, use_example_as_example, submitted_fields): if use_example_as_example: @@ -574,6 +574,7 @@ def _check_messages(service_id, template_id, upload_id, preview_row): service_id, template.id, db_template["version"], original_file_name ), template_id=template_id, + db_template=db_template ) @@ -617,6 +618,7 @@ def check_messages(service_id, template_id, upload_id, row_index=2): metadata_kwargs["sender_id"] = session["sender_id"] set_metadata_on_csv_upload(service_id, upload_id, **metadata_kwargs) + return render_template("views/check/ok.html", **data) @@ -649,8 +651,14 @@ def preview_job(service_id, template_id, upload_id, row_index=2): if data["errors"]: return render_template("views/check/column-errors.html", **data) + simplifed_template = get_template( + data.get('db_template', {}), + current_service, + ) + return render_template( - "views/check/preview.html", scheduled_for=session["scheduled_for"], **data + "views/check/preview.html", scheduled_for=session["scheduled_for"], **data, + simplifed_template=simplifed_template ) @@ -857,6 +865,7 @@ def _check_notification(service_id, template_id, exception=None): back_link=back_link, back_link_from_preview=back_link_from_preview, choose_time_form=choose_time_form, + db_template=db_template, **(get_template_error_dict(exception) if exception else {}), ) @@ -904,11 +913,20 @@ def preview_notification(service_id, template_id): ) session["scheduled_for"] = request.args.get("scheduled_for", "") + data = _check_notification(service_id, template_id) + db_template = data.get('db_template', None) + + simplifed_template = get_template( + db_template, + current_service, + ) return render_template( "views/notifications/preview.html", - **_check_notification(service_id, template_id), + **data, scheduled_for=session["scheduled_for"], + simplifed_template=simplifed_template, + recipient=recipient, ) diff --git a/app/templates/views/check/preview.html b/app/templates/views/check/preview.html index fda9bc5f6..f6fe2cb04 100644 --- a/app/templates/views/check/preview.html +++ b/app/templates/views/check/preview.html @@ -20,12 +20,15 @@ {% block maincolumn_content %} {{ page_header('Preview') }} +
+

Scheduled: {{ scheduled_for if scheduled_for else 'Now'}}

+

File: {{original_file_name}}

+

Template: {{template.name}}

+

From: {{ template.sender }}

+
+

Message

-

Scheduled: - {{ scheduled_for if scheduled_for else 'Now'}}

-

File: {{original_file_name}}

-

Template: {{template.name}}

- {{ template|string }} +
{{ simplifed_template|string }}
{% endif %} -

Placeholder: This message will be delivered to 400 phone numbers and will use a total of 800 message parts, leaving Washington DSHS with 249,200 message parts remaining.

+

Does everything look good?

{% set button_text %} - {{ "Scheduled" if scheduled_for else 'Send'}} + {{ "Schedule" if scheduled_for else 'Send'}} {% endset %} {{ usaButton({ "text": button_text }) }} diff --git a/app/templates/views/notifications/preview.html b/app/templates/views/notifications/preview.html index b0b66efe7..13ef103b0 100644 --- a/app/templates/views/notifications/preview.html +++ b/app/templates/views/notifications/preview.html @@ -5,7 +5,7 @@ {% from "components/components/button/macro.njk" import usaButton %} {% block service_page_title %} - {{ "Error" if error else "Preview of ‘{}’".format(template.name) }} + {{ "Error" if error else "Preview" }} {% endblock %} {% block backLink %} @@ -42,12 +42,15 @@ {% else %} {{ page_header('Preview') }} {% endif %} -

Message

+
+

Scheduled: {{ scheduled_for if scheduled_for else 'Now'}}

+

Template: {{template.name}}

+

From: {{ template.sender }}

+

To: {{ recipient }}

+
-

Scheduled: - {{ scheduled_for if scheduled_for else 'Now'}}

-

Template: {{template.name}}

- {{ template|string }} +

Message

+
{{ simplifed_template|string }}