diff --git a/app/assets/stylesheets/_grids.scss b/app/assets/stylesheets/_grids.scss index 818891e54..84cbbe94c 100644 --- a/app/assets/stylesheets/_grids.scss +++ b/app/assets/stylesheets/_grids.scss @@ -44,6 +44,11 @@ margin-top: $gutter-half; } +.top-gutter-1-3 { + @extend %top-gutter; + margin-top: $gutter / 3; +} + %bottom-gutter, .bottom-gutter { @extend %contain-floats; diff --git a/app/templates/views/templates/_template.html b/app/templates/views/templates/_template.html index 265597f68..f7a7d4d37 100644 --- a/app/templates/views/templates/_template.html +++ b/app/templates/views/templates/_template.html @@ -1,8 +1,16 @@ +{% from 'components/message-count-label.html' import message_count_label %} +
{% if template._template.archived %}

This template was deleted {{ template._template.updated_at|format_datetime_relative }}.

+ {% elif not current_user.has_permissions('send_messages', 'manage_api_keys', 'manage_templates', 'manage_service') %} +

+ If you need to send this + {{ message_count_label(1, template.template_type, suffix='') }} + or edit this template, contact your manager. +

{% else %}
diff --git a/tests/app/main/views/test_templates.py b/tests/app/main/views/test_templates.py index 6e7b04bed..3e1a30e57 100644 --- a/tests/app/main/views/test_templates.py +++ b/tests/app/main/views/test_templates.py @@ -110,22 +110,31 @@ def test_should_show_page_for_one_template( mock_get_service_template.assert_called_with(service_one['id'], template_id) -@pytest.mark.parametrize('permissions, links_to_be_shown', [ +@pytest.mark.parametrize('permissions, links_to_be_shown, permissions_warning_to_be_shown', [ ( ['view_activity'], - [] + [], + 'If you need to send this text message or edit this template, contact your manager.' + ), + ( + ['manage_api_keys'], + [], + None, ), ( ['manage_templates'], - ['.edit_service_template'] + ['.edit_service_template'], + None, ), ( ['send_messages'], - ['.send_messages', '.set_sender'] + ['.send_messages', '.set_sender'], + None, ), ( ['send_messages', 'manage_templates'], - ['.send_messages', '.set_sender', '.edit_service_template'] + ['.send_messages', '.set_sender', '.edit_service_template'], + None, ), ]) def test_should_be_able_to_view_a_template_with_links( @@ -138,6 +147,7 @@ def test_should_be_able_to_view_a_template_with_links( fake_uuid, permissions, links_to_be_shown, + permissions_warning_to_be_shown, ): active_user_with_permissions._permissions[service_one['id']] = permissions + ['view_activity'] client.login(active_user_with_permissions, mocker, service_one) @@ -161,6 +171,10 @@ def test_should_be_able_to_view_a_template_with_links( template_id=fake_uuid, ) + assert normalize_spaces(page.select_one('main p').text) == ( + permissions_warning_to_be_shown or 'To: phone number' + ) + def test_should_show_template_id_on_template_page( logged_in_client,