From de003a87223857da09e8ba7c27e9092a05de26ff Mon Sep 17 00:00:00 2001 From: Jonathan Bobel Date: Tue, 19 Mar 2024 12:10:43 -0400 Subject: [PATCH] Removed "failure" area so the user can get to all the statuses from the Dashboard Landing screen. A better way to view all statuses will come with a Dashboard redesign. --- app/templates/views/dashboard/_totals.html | 25 ++-------------------- tests/app/main/views/test_dashboard.py | 2 +- 2 files changed, 3 insertions(+), 24 deletions(-) diff --git a/app/templates/views/dashboard/_totals.html b/app/templates/views/dashboard/_totals.html index fab3b0435..87d6c22eb 100644 --- a/app/templates/views/dashboard/_totals.html +++ b/app/templates/views/dashboard/_totals.html @@ -17,30 +17,9 @@ Details - {% if show_failures %} - - {% if statistics['sms']['failed'] %} - - {{ "{:,}".format(statistics['sms']['failed']) }} - failed – {{ statistics['sms']['failed_percentage'] }}% - - {% else %} - No failures - {% endif %} - - {% endif %} + {# Removing the failures area for now, as the user can click on the above link to see all the details. + In the future state of the dashboard, the all statuses will be more apparent with data visualizations #} - diff --git a/tests/app/main/views/test_dashboard.py b/tests/app/main/views/test_dashboard.py index 37674e797..7d0ae9227 100644 --- a/tests/app/main/views/test_dashboard.py +++ b/tests/app/main/views/test_dashboard.py @@ -655,7 +655,7 @@ def test_should_not_show_recent_templates_on_dashboard_if_only_one_template_used expected_count = stats[0]["count"] assert expected_count == 50 assert normalize_spaces(page.select_one("#total-sms .big-number-smaller").text) == ( - "{} text messages sent".format(expected_count) + "{} text messages sent in the last seven days".format(expected_count) )