diff --git a/app/main/views/service_settings.py b/app/main/views/service_settings.py index 99146fbc4..3cd9f5585 100644 --- a/app/main/views/service_settings.py +++ b/app/main/views/service_settings.py @@ -532,11 +532,14 @@ def service_set_inbound_sms(service_id): @login_required @user_has_permissions('manage_service') def service_set_letters(service_id): - return redirect(url_for( - '.service_set_channel', - service_id=current_service.id, - channel='letter', - )) + return redirect( + url_for( + '.service_set_channel', + service_id=current_service.id, + channel='letter', + ), + code=301, + ) @main.route("/services//service-settings/set-", methods=['GET', 'POST']) diff --git a/tests/app/main/views/test_service_settings.py b/tests/app/main/views/test_service_settings.py index e55432f75..48ba3a810 100644 --- a/tests/app/main/views/test_service_settings.py +++ b/tests/app/main/views/test_service_settings.py @@ -2548,7 +2548,7 @@ def test_old_set_letters_page_redirects( client_request.get( 'main.service_set_letters', service_id=SERVICE_ONE_ID, - _expected_status=302, + _expected_status=301, _expected_redirect=url_for( 'main.service_set_channel', service_id=SERVICE_ONE_ID,