mirror of
https://github.com/GSA/notifications-admin.git
synced 2026-08-23 15:57:23 -04:00
Rename ‘Send a message’ to ‘Templates’ in basic view
From Karl: > Templates – this should be consistent with Admin view. Users may > switch from Basic to Admin view (or vice versa), they will also > interact with users who have a different view or permissions to them. > Neither should have to learn new interfaces and language if possible. > ‘Send a message’ was a nice, active label – but Notify options aren’t > usually actions. If we’re going to change this we should be consistent > across both Admin and Basic views. > For the same reason, I have rejected ‘see’, ‘search’ and ‘view sent > messages’. It will be interesting to see in user testing whether users > read ‘sent messages’ as ‘send messages’.
This commit is contained in:
@@ -122,7 +122,7 @@ def test_invite_goes_in_session(
|
||||
|
||||
@pytest.mark.parametrize('user, landing_page_title', [
|
||||
(active_user_with_permissions, 'Dashboard'),
|
||||
(active_caseworking_user, 'Choose a template'),
|
||||
(active_caseworking_user, 'Templates'),
|
||||
])
|
||||
def test_accepting_invite_removes_invite_from_session(
|
||||
client_request,
|
||||
|
||||
@@ -2459,7 +2459,7 @@ def test_preview_basic_view(
|
||||
with client_request.session_transaction() as session:
|
||||
assert session['basic'] is True
|
||||
|
||||
assert page.h1.text.strip() == 'Choose a template'
|
||||
assert page.h1.text.strip() == 'Templates'
|
||||
page.select('.navigation-service-basic-view-preview')
|
||||
assert normalize_spaces(page.select_one('.navigation-service').text) == (
|
||||
'service one '
|
||||
|
||||
@@ -70,7 +70,7 @@ from tests.conftest import single_letter_contact_block
|
||||
),
|
||||
(
|
||||
active_caseworking_user,
|
||||
'Choose a template',
|
||||
'Templates',
|
||||
{},
|
||||
['Text message', 'Email'],
|
||||
[
|
||||
@@ -82,7 +82,7 @@ from tests.conftest import single_letter_contact_block
|
||||
),
|
||||
(
|
||||
active_caseworking_user,
|
||||
'Choose a template',
|
||||
'Templates',
|
||||
{'template_type': 'email'},
|
||||
['All', 'Text message'],
|
||||
['email_template_one', 'email_template_two'],
|
||||
|
||||
Reference in New Issue
Block a user