fix: move secondary message outside banner

cds-snc · Dec 3, 2024 · e7ad1de · e7ad1de
1 parent 82b44e0
commit e7ad1de
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/app/templates/partials/check/too-many-email-messages.html b/app/templates/partials/check/too-many-email-messages.html
@@ -1,3 +1,4 @@
+{% from "components/banner.html" import banner_wrapper %}
 {% from "components/links.html" import content_link %}
 
 <p>
@@ -13,8 +14,10 @@
       </p>
 
     {% elif send_exceeds_daily_limit or recipients.more_rows_than_can_send %}
-      {{ _("To request a daily limit above {} emails, {}").format(current_service.message_limit, content_link(_("contact us"), url_for('main.contact'), is_external_link=true)) }}
-
+      {% call banner_wrapper(type='dangerous') %}
+        {{ _("To request a daily limit above {} emails, {}").format(current_service.message_limit, content_link(_("contact us"), url_for('main.contact'), is_external_link=true)) }}
+      {% endcall %}
+
       <h2 class="heading-medium">{{ _('You cannot send all these email messages today') }}</h2>
       <p data-testid="exceeds-daily">
         {{ _("You can try sending these messages after {} Eastern Time. Check {}.").format(time_to_reset[current_lang],

diff --git a/app/templates/views/check/column-errors.html b/app/templates/views/check/column-errors.html
@@ -171,9 +171,7 @@ <h2 class="heading-medium">{{ _('You cannot send all these text messages today')
           {% include "partials/check/too-many-email-messages.html" %}
         {% endcall %}
       {% elif recipients.more_rows_than_can_send or  send_exceeds_annual_limit %}
-          {% call banner_wrapper(type='dangerous') %}
-            {% include "partials/check/too-many-email-messages.html" %}
-          {% endcall %}           
+          {% include "partials/check/too-many-email-messages.html" %}
       {% endif %}
   </div>