Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Display intervention banner on specific pages for User Research #2378

Merged
merged 3 commits into from
Mar 7, 2022

Conversation

chao-xian
Copy link
Contributor

@chao-xian chao-xian commented Mar 4, 2022

What / Why

Adds the intervention banner to specific pages that links to a tree test that User Research want to conduct.

Trello card

⚠️ This repo is Continuously Deployed: make sure you follow the guidance ⚠️

Screenshots

Before

www gov uk_register-for-self-assessment(iPhone SE)

After

localhost_3090_register-for-self-assessment(iPhone SE)

@govuk-ci govuk-ci temporarily deployed to government-f-recruitmen-vke29i March 4, 2022 16:36 Inactive
@chao-xian chao-xian changed the title Add test helper to generate pages with specific base_path Display intervention banner on specific pages for User Research Mar 4, 2022
@chao-xian chao-xian force-pushed the recruitment-banner branch from eb209f7 to 7596e60 Compare March 4, 2022 16:50
@govuk-ci govuk-ci temporarily deployed to government-f-recruitmen-vke29i March 4, 2022 16:51 Inactive
<% if @content_item.show_phase_banner? %>
<% if @content_item.show_study_banner? %>
<%= render "govuk_publishing_components/components/intervention", {
suggestion_text: "Help improve GOV.UK",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we have these in i18n anywhere?

Copy link
Contributor

@Tetrino Tetrino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems good - we should probably add the strings to en.yml at some point if we don't have any previous examples of them being hardcoded.

@chao-xian
Copy link
Contributor Author

chao-xian commented Mar 7, 2022

@Tetrino I had that same option in Collections but the gem for GOV.UK Translations checks that you have keys in all of the lang files, and fails it as a lint error unless all the 60ish files have keys. Given that the banner is temporary and will only be in English I'm ok with it being hardcoded.

@chao-xian chao-xian merged commit 5a3bdbe into main Mar 7, 2022
@chao-xian chao-xian deleted the recruitment-banner branch March 7, 2022 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants