Put template type navigation in correct order

We always talk about the things you can send using Notify as _emails,
text messages and letters_, in that order.

The navigation should reflect this.
This commit is contained in:
Chris Hill-Scott
2019-02-22 17:11:12 +00:00
parent 4b8788aea8
commit 8aeb203124
3 changed files with 9 additions and 9 deletions

View File

@@ -39,8 +39,8 @@ class Service():
} }
TEMPLATE_TYPES = ( TEMPLATE_TYPES = (
'sms',
'email', 'email',
'sms',
'letter', 'letter',
) )

View File

@@ -55,7 +55,7 @@ def _template(template_type, name, parent=None, template_id=None):
'Templates', 'Templates',
[], [],
{}, {},
['Text message', 'Email', 'Letter'], ['Email', 'Text message', 'Letter'],
[ [
'folder_one 2 folders', 'folder_one 2 folders',
'folder_one / folder_one_one 1 template, 1 folder', 'folder_one / folder_one_one 1 template, 1 folder',
@@ -132,7 +132,7 @@ def _template(template_type, name, parent=None, template_id=None):
'Templates / folder_one', 'Templates / folder_one',
[{'template_type': 'all'}], [{'template_type': 'all'}],
{'template_folder_id': PARENT_FOLDER_ID}, {'template_folder_id': PARENT_FOLDER_ID},
['Text message', 'Email', 'Letter'], ['Email', 'Text message', 'Letter'],
[ [
'folder_one_one 1 template, 1 folder', 'folder_one_one 1 template, 1 folder',
'folder_one_one / folder_one_one_one 1 template', 'folder_one_one / folder_one_one_one 1 template',
@@ -193,7 +193,7 @@ def _template(template_type, name, parent=None, template_id=None):
{'template_type': 'all', 'template_folder_id': PARENT_FOLDER_ID}, {'template_type': 'all', 'template_folder_id': PARENT_FOLDER_ID},
], ],
{'template_folder_id': CHILD_FOLDER_ID}, {'template_folder_id': CHILD_FOLDER_ID},
['Text message', 'Email', 'Letter'], ['Email', 'Text message', 'Letter'],
[ [
'folder_one_one_one 1 template', 'folder_one_one_one 1 template',
'folder_one_one_one / sms_template_nested Text message template', 'folder_one_one_one / sms_template_nested Text message template',
@@ -219,7 +219,7 @@ def _template(template_type, name, parent=None, template_id=None):
{'template_type': 'all', 'template_folder_id': CHILD_FOLDER_ID}, {'template_type': 'all', 'template_folder_id': CHILD_FOLDER_ID},
], ],
{'template_folder_id': GRANDCHILD_FOLDER_ID}, {'template_folder_id': GRANDCHILD_FOLDER_ID},
['Text message', 'Email', 'Letter'], ['Email', 'Text message', 'Letter'],
[ [
'sms_template_nested Text message template', 'sms_template_nested Text message template',
], ],
@@ -254,7 +254,7 @@ def _template(template_type, name, parent=None, template_id=None):
'Templates / folder_two', 'Templates / folder_two',
[{'template_type': 'all'}], [{'template_type': 'all'}],
{'template_folder_id': FOLDER_TWO_ID}, {'template_folder_id': FOLDER_TWO_ID},
['Text message', 'Email', 'Letter'], ['Email', 'Text message', 'Letter'],
[], [],
[], [],
[], [],

View File

@@ -96,7 +96,7 @@ def test_should_show_add_template_form_if_service_has_folder_permission(
active_user_view_permissions, active_user_view_permissions,
'Templates', 'Templates',
{}, {},
['Text message', 'Email', 'Letter'], ['Email', 'Text message', 'Letter'],
[ [
'sms_template_one', 'sms_template_one',
'sms_template_two', 'sms_template_two',
@@ -124,14 +124,14 @@ def test_should_show_add_template_form_if_service_has_folder_permission(
active_user_view_permissions, active_user_view_permissions,
'Templates', 'Templates',
{'template_type': 'letter'}, {'template_type': 'letter'},
['All', 'Text message', 'Email'], ['All', 'Email', 'Text message'],
['letter_template_one', 'letter_template_two'], ['letter_template_one', 'letter_template_two'],
), ),
( (
active_caseworking_user, active_caseworking_user,
'Templates', 'Templates',
{}, {},
['Text message', 'Email', 'Letter'], ['Email', 'Text message', 'Letter'],
[ [
'sms_template_one', 'sms_template_one',
'sms_template_two', 'sms_template_two',