From 8b06938aa0a36f808f27de61722363e5fb0bd0c2 Mon Sep 17 00:00:00 2001 From: Ben Thorner Date: Wed, 17 Feb 2021 15:23:33 +0000 Subject: [PATCH] Restore original wording for travel advice emails This has been requested by FCDO [1], and agreed with GOV.UK Notify, who originally asked for the word "alert" to be removed so that it did not conflict with the upcoming emergency alert pages on GOV.UK. Note the original change was applied via the default text changing in GOV.UK Publishing Components [2]. [1]: https://govuk.zendesk.com/agent/tickets/4417745 [2]: https://github.com/alphagov/govuk_publishing_components/commit/36953fd979db6c144ba0fd0245ed414066fc88b0 --- app/views/content_items/travel_advice.html.erb | 5 ++++- test/integration/travel_advice_test.rb | 2 +- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/app/views/content_items/travel_advice.html.erb b/app/views/content_items/travel_advice.html.erb index 23ae975bd..cebba6237 100644 --- a/app/views/content_items/travel_advice.html.erb +++ b/app/views/content_items/travel_advice.html.erb @@ -57,7 +57,10 @@ <%= render "govuk_publishing_components/components/contents_list", contents: @content_item.part_link_elements, underline_links: true %> - <%= render 'govuk_publishing_components/components/subscription_links', email_signup_link: @content_item.email_signup_link, feed_link: @content_item.feed_link %> + <%= render 'govuk_publishing_components/components/subscription_links', + email_signup_link: @content_item.email_signup_link, + email_signup_link_text: "Get email alerts", + feed_link: @content_item.feed_link %> diff --git a/test/integration/travel_advice_test.rb b/test/integration/travel_advice_test.rb index 9ca9a6384..cb0011417 100644 --- a/test/integration/travel_advice_test.rb +++ b/test/integration/travel_advice_test.rb @@ -11,7 +11,7 @@ class TravelAdviceTest < ActionDispatch::IntegrationTest assert page.has_css?("title", visible: false, text: @content_item["title"]) assert_has_component_title(@content_item["details"]["country"]["name"]) - assert page.has_css?("a[href=\"#{@content_item['details']['email_signup_link']}\"]", text: "Get emails") + assert page.has_css?("a[href=\"#{@content_item['details']['email_signup_link']}\"]", text: "Get email alerts") assert page.has_css?("a[href=\"#{@content_item['base_path']}.atom\"]", text: "Subscribe to feed") assert page.has_css?(".part-navigation li", count: @content_item["details"]["parts"].size + 1)