mirror of
https://github.com/GSA/notifications-admin.git
synced 2026-07-12 17:34:16 -04:00
Remove question about which features people use
This is an optional question. We don’t really use the answer to it now that we have some adoption of all these features.
This commit is contained in:
@@ -584,9 +584,6 @@ class RequestToGoLiveForm(StripWhitespaceForm):
|
||||
'Will the number of messages increase and when will that start?',
|
||||
validators=[DataRequired(message='Can’t be empty')]
|
||||
)
|
||||
method_one_off = BooleanField('One at a time')
|
||||
method_upload = BooleanField('Upload a spreadsheet of recipients')
|
||||
method_api = BooleanField('Integrate with the GOV.UK Notify API')
|
||||
|
||||
|
||||
class ProviderForm(StripWhitespaceForm):
|
||||
|
||||
@@ -224,7 +224,6 @@ def submit_request_to_go_live(service_id):
|
||||
'\nAgreement signed: {}'
|
||||
'\nChannel: {}\nStart date: {}\nStart volume: {}'
|
||||
'\nPeak volume: {}'
|
||||
'\nFeatures: {}'
|
||||
'\n'
|
||||
'\n---'
|
||||
'\n'
|
||||
@@ -248,11 +247,6 @@ def submit_request_to_go_live(service_id):
|
||||
form.start_date.data,
|
||||
form.start_volume.data,
|
||||
form.peak_volume.data,
|
||||
formatted_list(filter(None, (
|
||||
'one off' if form.method_one_off.data else None,
|
||||
'file upload' if form.method_upload.data else None,
|
||||
'API' if form.method_api.data else None,
|
||||
)), before_each='', after_each=''),
|
||||
service_id=current_service.id,
|
||||
organisation=AgreementInfo.from_current_user().owner,
|
||||
service_name=current_service.name,
|
||||
|
||||
@@ -28,11 +28,6 @@
|
||||
{{ textbox(form.start_volume, width='1-1', hint='For example, ‘1,000 per month’.') }}
|
||||
{{ textbox(form.peak_volume, width='1-1', hint='For example, ‘Messages will increase to 20,000 per month in January’.') }}
|
||||
</div>
|
||||
{{ checkbox_group('How are you going to send messages?', [
|
||||
form.method_one_off,
|
||||
form.method_upload,
|
||||
form.method_api
|
||||
]) }}
|
||||
<p class="bottom-gutter">
|
||||
By requesting to go live you’re agreeing to our <a href="{{ url_for('.terms') }}">terms of use</a>.
|
||||
</p>
|
||||
|
||||
@@ -698,14 +698,6 @@ def test_should_show_request_to_go_live(
|
||||
assert normalize_spaces(
|
||||
page.select_one('label[for=channel_{}]'.format(channel)).text
|
||||
) == label
|
||||
for feature, label in (
|
||||
('one_off', 'One at a time'),
|
||||
('upload', 'Upload a spreadsheet of recipients'),
|
||||
('api', 'Integrate with the GOV.UK Notify API'),
|
||||
):
|
||||
assert normalize_spaces(
|
||||
page.select_one('label[for=method_{}]'.format(feature)).text
|
||||
) == label
|
||||
|
||||
|
||||
@freeze_time("2012-12-21")
|
||||
@@ -729,9 +721,6 @@ def test_should_redirect_after_request_to_go_live(
|
||||
'start_date': '01/01/2017',
|
||||
'start_volume': '100,000',
|
||||
'peak_volume': '2,000,000',
|
||||
'method_one_off': 'y',
|
||||
'method_upload': 'y',
|
||||
'method_api': 'y',
|
||||
},
|
||||
_follow_redirects=True
|
||||
)
|
||||
@@ -753,7 +742,6 @@ def test_should_redirect_after_request_to_go_live(
|
||||
'Start date: 01/01/2017\n'
|
||||
'Start volume: 100,000\n'
|
||||
'Peak volume: 2,000,000\n'
|
||||
'Features: one off, file upload and API\n'
|
||||
'\n'
|
||||
'---\n'
|
||||
'{}\tNone\tservice one\tTest User\ttest@user.gov.uk\t-\t21/12/2012'
|
||||
|
||||
Reference in New Issue
Block a user