Decided not to use the template and include the USWDS alert one-by-one

This commit is contained in:
Jonathan Bobel
2023-10-06 11:31:28 -04:00
parent fd196c6534
commit af2da54ffc
5 changed files with 226 additions and 199 deletions
@@ -1,7 +1,9 @@
<h1 class='banner-title'> <div class="usa-alert usa-alert--error" role="alert">
Message too long <div class="usa-alert__body">
</h1> <h1 class="usa-alert__heading">Message too long</h1>
<p class="usa-body"> <p class="usa-alert__text">
Text messages cannot be longer than {{ SMS_CHAR_COUNT_LIMIT }} characters. Text messages cannot be longer than {{ SMS_CHAR_COUNT_LIMIT }} characters.
Your message is {{ template.content_count }} characters. Your message is {{ template.content_count }} characters.
</p> </p>
</div>
</div>
@@ -1,4 +1,6 @@
<h1 class='banner-title' data-module="track-error" data-error-type="Trial mode: bad recipients" data-error-label="{{ upload_id }}"> <div class="usa-alert usa-alert--error" role="alert">
<div class="usa-alert__body">
<h1 class="usa-alert__heading">
You cannot send to You cannot send to
{{ 'this' if count_of_recipients == 1 else 'these' }} {{ 'this' if count_of_recipients == 1 else 'these' }}
{{ template_type_label }} {{ template_type_label }}
@@ -6,7 +8,9 @@
{{ 'es' if 'email address' == template_type_label else 's' }} {{ 'es' if 'email address' == template_type_label else 's' }}
{%- endif %} {%- endif %}
</h1> </h1>
<p class="usa-body"> <p class="usa-alert__text">
In <a class="usa-link" href="{{ url_for('.trial_mode_new') }}">trial mode</a> you can only In <a class="usa-link" href="{{ url_for('.trial_mode_new') }}">trial mode</a> you can only
send to yourself and members of your team send to yourself and members of your team
</p> </p>
</div>
</div>
@@ -1,6 +1,8 @@
<h1 class='banner-title' data-module="track-error" data-error-type="File previously sent" data-error-label="{{ upload_id }}"> <div class="usa-alert usa-alert--error" role="alert">
These messages have already been sent today <div class="usa-alert__body">
</h1> <h1 class="usa-alert__heading">These messages have already been sent today</h1>
<p class="usa-body"> <p class="usa-alert__text">
If you need to resend them, rename the file and upload it again. If you need to resend them, rename the file and upload it again.
</p> </p>
</div>
</div>
@@ -1,11 +1,13 @@
<h1 class='banner-title' data-module="track-error" data-error-type="Trial mode: too many recipients" data-error-label="{{ upload_id }}"> <div class="usa-alert usa-alert--error" role="alert">
<div class="usa-alert__body">
<h1 class="usa-alert__heading">
{% if original_file_name %} {% if original_file_name %}
Too many recipients Too many recipients
{% else %} {% else %}
Daily limit reached Daily limit reached
{% endif %} {% endif %}
</h1> </h1>
<p class="usa-body"> <p class="usa-alert__text">
You can only send {{ current_service.message_limit|format_thousands }} messages per day You can only send {{ current_service.message_limit|format_thousands }} messages per day
{%- if current_service.trial_mode %} {%- if current_service.trial_mode %}
in <a class="usa-link" href="{{ url_for('.trial_mode_new')}}">trial mode</a> in <a class="usa-link" href="{{ url_for('.trial_mode_new')}}">trial mode</a>
@@ -13,7 +15,7 @@
. .
</p> </p>
{% if original_file_name %} {% if original_file_name %}
<p class="usa-body"> <p class="usa-alert__text">
{% if current_service.message_limit != remaining_messages %} {% if current_service.message_limit != remaining_messages %}
You can still send {{ remaining_messages|format_thousands }} messages today, but You can still send {{ remaining_messages|format_thousands }} messages today, but
{% endif %} {% endif %}
@@ -21,3 +23,8 @@
{{ count_of_recipients|recipient_count(template.template_type) }}. {{ count_of_recipients|recipient_count(template.template_type) }}.
</p> </p>
{% endif %} {% endif %}
</div>
</div>
+50 -38
View File
@@ -4,8 +4,6 @@
{% from "components/table.html" import list_table, field, text_field, index_field, hidden_field_heading %} {% from "components/table.html" import list_table, field, text_field, index_field, hidden_field_heading %}
{% from "components/file-upload.html" import file_upload %} {% from "components/file-upload.html" import file_upload %}
{% from "components/components/back-link/macro.njk" import usaBackLink %} {% from "components/components/back-link/macro.njk" import usaBackLink %}
{% from "components/components/alert/macro.njk" import usaAlert %}
{% block service_page_title %} {% block service_page_title %}
Error Error
@@ -22,79 +20,93 @@
{% if recipients.too_many_rows %} {% if recipients.too_many_rows %}
<h1 class='banner-title' data-module="track-error" data-error-type="Too many rows" data-error-label="{{ upload_id }}"> <div class="usa-alert usa-alert--error" role="alert">
Your file has too many rows <div class="usa-alert__body">
</h1> <h1 class="usa-alert__heading banner-title" data-module="track-error" data-error-type="Too many rows" data-error-label="{{ upload_id }}">Your file has too many rows</h1>
<p> <p class="usa-alert__text">
Notify can process up to Notify can process up to
{{ "{:,}".format(recipients.max_rows) }} rows at once. Your {{ "{:,}".format(recipients.max_rows) }} rows at once. Your
file has {{ "{:,}".format(recipients|length) }} rows. file has {{ "{:,}".format(recipients|length) }} rows.
</p> </p>
</div>
</div>
{% elif not count_of_recipients %} {% elif not count_of_recipients %}
<h1 class='banner-title' data-module="track-error" data-error-type="No rows" data-error-label="{{ upload_id }}"> <div class="usa-alert usa-alert--error" role="alert">
Your file is missing some rows <div class="usa-alert__body">
<h1 class="usa-alert__heading banner-title" data-module="track-error" data-error-type="No rows"
data-error-label="{{ upload_id }}">Your file is missing some rows
</h1> </h1>
{% if recipients.missing_column_headers %} {% if recipients.missing_column_headers %}
<p> <p class="usa-alert-text">
It needs at least one row of data, and {{ recipients.missing_column_headers | sort() | formatted_list( It needs at least one row of data, and {{ recipients.missing_column_headers | sort() | formatted_list(
prefix='a column called', prefix='a column called',
prefix_plural='columns called' prefix_plural='columns called'
) }}. ) }}.
</p> </p>
{% else %} {% else %}
<p> <p class="usa-alert-text">
It needs at least one row of data. It needs at least one row of data.
</p> </p>
{% endif %} {% endif %}
</div>
</div>
{% elif not recipients.has_recipient_columns %} {% elif not recipients.has_recipient_columns %}
<h1 class='banner-title' data-module="track-error" data-error-type="Missing recipient columns" data-error-label="{{ upload_id }}"> <div class="usa-alert usa-alert--error" role="alert">
Theres a problem with your column names <div class="usa-alert__body">
</h1> <h1 class="usa-alert__heading banner-title" data-module="track-error" data-error-type="Missing recipient columns"
<p> data-error-label="{{ upload_id }}">Theres a problem with your column names</h1>
<p class="usa-alert-text">
Your file needs a column called {{ first_recipient_column }}. Your file needs a column called {{ first_recipient_column }}.
</p> </p>
<p> <p class="usa-alert-text">
Right now it has {{ recipients.column_headers | formatted_list( Right now it has {{ recipients.column_headers | formatted_list(
prefix='one column, called ', prefix='one column, called ',
prefix_plural='columns called ' prefix_plural='columns called '
) }}. ) }}.
</p> </p>
</div>
</div>
{% elif recipients.duplicate_recipient_column_headers %} {% elif recipients.duplicate_recipient_column_headers %}
{% set duplicate_recipient_column_headers = recipients.duplicate_recipient_column_headers <div class="usa-alert usa-alert--error" role="alert">
| formatted_list( <div class="usa-alert__body">
<h1 class="usa-alert__heading banner-title" data-module="track-error" data-error-type="Duplicate recipient columns"
data-error-label="{{ upload_id }}">Theres a problem with your column names</h1>
<p class="usa-alert__text">
We found more than one column called {{ (
recipients.duplicate_recipient_column_headers
) | formatted_list(
conjunction='or', conjunction='or',
prefix='', prefix='',
prefix_plural='' prefix_plural=''
) %} ) }}.
</p>
{{ usaAlert({ <p class="usa-alert__text">
"heading": "Theres a problem with your column names", Delete or rename one of these columns and try again.
"text": "We found more than one column called " + duplicate_recipient_column_headers + ". Delete or rename one of these columns and try again.", </p>
"type": "error", </div>
}) </div>
}}
{% elif recipients.missing_column_headers %} {% elif recipients.missing_column_headers %}
{% set missing_column_headers = recipients.missing_column_headers | formatted_list( <div class="usa-alert usa-alert--error" role="alert">
<div class="usa-alert__body">
<h1 class="usa-alert__heading banner-title" data-module="track-error" data-error-type="Missing placeholder columns"
data-error-label="{{ upload_id }}">Your column names need to match the double brackets in your template</h1>
<p class="usa-alert__text">
Your file is missing {{ recipients.missing_column_headers | formatted_list(
conjunction='and', conjunction='and',
prefix='a column called ', prefix='a column called ',
prefix_plural='columns called ' prefix_plural='columns called '
)%} ) }}.
</p>
{{ usaAlert({ </div>
"heading": "Your column names need to match the double brackets in your template", </div>
"text": "Your file is missing " + missing_column_headers + ".",
"type": "error",
})
}}
{% elif sent_previously %} {% elif sent_previously %}
@@ -135,7 +147,8 @@
{% if not request.args.from_test %} {% if not request.args.from_test %}
{% set column_headers = recipients._raw_column_headers if recipients.duplicate_recipient_column_headers else recipients.column_headers %} {% set column_headers = recipients._raw_column_headers if recipients.duplicate_recipient_column_headers else
recipients.column_headers %}
<h2 class="font-body-lg" id="file-preview">{{ original_file_name }}</h2> <h2 class="font-body-lg" id="file-preview">{{ original_file_name }}</h2>
@@ -181,8 +194,7 @@
<p class="table-show-more-link"> <p class="table-show-more-link">
Only showing the first {{ count_of_displayed_recipients }} rows Only showing the first {{ count_of_displayed_recipients }} rows
</p> </p>
{% elif count_of_displayed_recipients < count_of_recipients %} {% elif count_of_displayed_recipients < count_of_recipients %} <p class="table-show-more-link">
<p class="table-show-more-link">
{% if row_errors and not recipients.missing_column_headers %} {% if row_errors and not recipients.missing_column_headers %}
Only showing the first {{ count_of_displayed_recipients }} rows with errors Only showing the first {{ count_of_displayed_recipients }} rows with errors
{% else %} {% else %}