Rename ‘caseworker’ to ‘basic view’

‘Caseworker’ was a bad name because it:
- suggested that Notify might be expanding into case management
- may or may not map to someone’s actual role, in a confusing way (this
  is why ‘manager’ is also a bad name)

‘Basic view’ is the best name we could come up with because:
- it describes the purpose of feature, not the user
- a ‘view’ changes what you can _see_ as much as it changes what you can
  do

Admin remains a good word – in research users self-describe their use
of Notify in using it. This commit makes the name ‘admin view’ to match
‘basic view’.

This also means we can hide the legend for this fieldset because the
choices are self-explanatory.
This commit is contained in:
Chris Hill-Scott
2018-07-09 09:54:54 +01:00
parent b1aac841f4
commit 5794a54385
4 changed files with 6 additions and 6 deletions

View File

@@ -303,8 +303,8 @@ class CaseworkingPermissionsForm(AbstractPermissionsForm):
user_type = RadioField(
'User type',
choices=[
('caseworker', 'Caseworker'),
('admin', 'Admin'),
('caseworker', 'Basic view'),
('admin', 'Admin view'),
],
)

View File

@@ -3,7 +3,7 @@
{% if 'caseworking' in current_service.permissions %}
<div class="conditional-radios" data-module='conditional-radios'>
{% call radios_wrapper(form.user_type) %}
{% call radios_wrapper(form.user_type, hide_legend=True) %}
{% for option in form.user_type %}
<div class="bottom-gutter-1-3">

View File

@@ -366,7 +366,7 @@
</li>
<li class="bottom-gutter">
<a href="{{ url_for('.service_switch_caseworking', service_id=current_service.id) }}" class="button">
{{ 'Stop granting of caseworking permission' if 'caseworking' in current_service.permissions else 'Allow granting of caseworking permission' }}
{{ 'Stop basic view' if 'caseworking' in current_service.permissions else 'Allow basic view' }}
</a>
</li>
<li class="bottom-gutter">

View File

@@ -45,9 +45,9 @@ def get_service_settings_page(
'.service_switch_can_upload_document', {}, 'Allow to upload documents'),
({'permissions': []},
'.service_switch_caseworking', {}, 'Allow granting of caseworking permission'),
'.service_switch_caseworking', {}, 'Allow basic view'),
({'permissions': ['caseworking']},
'.service_switch_caseworking', {}, 'Stop granting of caseworking permission'),
'.service_switch_caseworking', {}, 'Stop basic view'),
({'permissions': ['sms']}, '.service_set_inbound_number', {'set_inbound_sms': True}, 'Allow inbound sms'),