Show letters in basic view

We were taking letters away from basic view because there’s no one-off
sending of letters.

But now that basic view isn’t an opt-in thing but something we’re
rolling out to all users who only have the ‘send’ permission it feels
like too drastic a change to remove letters.
This commit is contained in:
Chris Hill-Scott
2018-08-07 11:45:19 +01:00
parent 3e14e5e52b
commit 5108187cab
2 changed files with 5 additions and 6 deletions

View File

@@ -111,10 +111,7 @@ def start_tour(service_id, template_id):
def choose_template(service_id, template_type='all'): def choose_template(service_id, template_type='all'):
templates = service_api_client.get_service_templates(service_id)['data'] templates = service_api_client.get_service_templates(service_id)['data']
letters_available = ( letters_available = current_service.has_permission('letter')
current_service.has_permission('letter') and
current_user.has_permissions('view_activity')
)
available_template_types = list(filter(None, ( available_template_types = list(filter(None, (
'email', 'email',

View File

@@ -74,19 +74,21 @@ from tests.conftest import single_letter_contact_block
active_caseworking_user, active_caseworking_user,
'Templates', 'Templates',
{}, {},
['Text message', 'Email'], ['Text message', 'Email', 'Letter'],
[ [
'sms_template_one', 'sms_template_one',
'sms_template_two', 'sms_template_two',
'email_template_one', 'email_template_one',
'email_template_two', 'email_template_two',
'letter_template_one',
'letter_template_two',
], ],
), ),
( (
active_caseworking_user, active_caseworking_user,
'Templates', 'Templates',
{'template_type': 'email'}, {'template_type': 'email'},
['All', 'Text message'], ['All', 'Text message', 'Letter'],
['email_template_one', 'email_template_two'], ['email_template_one', 'email_template_two'],
), ),
] ]