From c5db8475430495ee82e5377f18ac524d2a1a00cb Mon Sep 17 00:00:00 2001 From: Katie Smith Date: Wed, 2 Feb 2022 16:44:32 +0000 Subject: [PATCH] Change both existing branding URLs This changes the URLs for someone to request new email or letter branding to match the new URLs we've agreed for the new email branding changes. The old URLs are still in place for now too to keep backwards compatibility. --- app/main/views/service_settings.py | 2 ++ tests/app/main/views/test_service_settings.py | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/app/main/views/service_settings.py b/app/main/views/service_settings.py index 2aeb4a9e5..c0e64fc04 100644 --- a/app/main/views/service_settings.py +++ b/app/main/views/service_settings.py @@ -1154,6 +1154,7 @@ def create_email_branding_zendesk_ticket(form_option_selected, detail=None): @main.route("/services//branding-request/email", methods=['GET', 'POST']) +@main.route("/services//service-settings/email-branding", methods=['GET', 'POST']) @user_has_permissions('manage_service') def email_branding_request(service_id): form = BrandingOptions(current_service, branding_type='email') @@ -1260,6 +1261,7 @@ def email_branding_something_else(service_id): @main.route("/services//branding-request/letter", methods=['GET', 'POST']) +@main.route("/services//service-settings/letter-branding", methods=['GET', 'POST']) @user_has_permissions('manage_service') def letter_branding_request(service_id): form = BrandingOptions(current_service, branding_type='letter') diff --git a/tests/app/main/views/test_service_settings.py b/tests/app/main/views/test_service_settings.py index 9e5f4a06a..36639693b 100644 --- a/tests/app/main/views/test_service_settings.py +++ b/tests/app/main/views/test_service_settings.py @@ -5761,7 +5761,7 @@ def test_service_settings_links_to_branding_request_page_for_letters( page = client_request.get( '.service_settings', service_id=SERVICE_ONE_ID ) - assert len(page.find_all('a', attrs={'href': '/services/{}/branding-request/letter'.format(SERVICE_ONE_ID)})) == 1 + assert len(page.find_all('a', attrs={'href': f'/services/{SERVICE_ONE_ID}/service-settings/letter-branding'})) == 1 def test_show_service_data_retention(