darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-08-24 00:07:02 -04:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #648 from alphagov/redirect-after-saving-template

Browse Source 
Redirect to single template view after edit
This commit is contained in:
Chris Hill-Scott
2016-06-06 11:06:24 +01:00
parent 2c17261795 7884f7870b
commit f71780ee27
2 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app/main/views/templates.py
View File

@@ -154,9 +154,9 @@ def edit_service_template(service_id, template_id):
raise e raise e
else: else:
return redirect(url_for( return redirect(url_for(
'.choose_template', '.view_template',
service_id=service_id, service_id=service_id,
template_type=template['template_type'] template_id=template_id
)) ))
return render_template( return render_template(
'views/edit-{}-template.html'.format(template['template_type']), 'views/edit-{}-template.html'.format(template['template_type']),

6
tests/app/main/views/test_templates.py
View File

@@ -63,7 +63,7 @@ def test_should_redirect_when_saving_a_template(app_,
assert response.status_code == 302 assert response.status_code == 302
assert response.location == url_for( assert response.location == url_for(
'.choose_template', service_id=service_id, template_type='sms', _external=True) '.view_template', service_id=service_id, template_id=template_id, _external=True)
mock_update_service_template.assert_called_with( mock_update_service_template.assert_called_with(
template_id, name, 'sms', content, service_id, None) template_id, name, 'sms', content, service_id, None)
@@ -210,9 +210,9 @@ def test_should_redirect_when_saving_a_template_email(app_,
template_id=template_id), data=data) template_id=template_id), data=data)
assert response.status_code == 302 assert response.status_code == 302
assert response.location == url_for( assert response.location == url_for(
'.choose_template', '.view_template',
service_id=service_id, service_id=service_id,
template_type='email', template_id=template_id,
_external=True) _external=True)
mock_update_service_template.assert_called_with( mock_update_service_template.assert_called_with(
template_id, name, 'email', content, service_id, subject) template_id, name, 'email', content, service_id, subject)