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

Add AB Test for Explore Super Menu Header #6164

Merged
merged 2 commits into from
Jul 15, 2021
Merged

Add AB Test for Explore Super Menu Header #6164

merged 2 commits into from
Jul 15, 2021

Conversation

chao-xian
Copy link
Contributor

@chao-xian chao-xian commented Jun 22, 2021

Static has a new header that the GOV.UK Explore team want to AB test as part of their beta.

See alphagov/static#2515

Trello https://trello.com/c/LUhrajXF/323-prepare-the-new-menu-a-b-test-for-sections-of-govuk

Visual changes

Screen.Recording.2021-06-28.at.9.35.34.pm.mov

For the ContentItemsController to allow views to display a different template from Static that includes the new header if users are in the B variant.
Enables the AB Test to show users in the B variant the new header menu.

WIP
@chao-xian chao-xian marked this pull request as ready for review July 2, 2021 09:57
@chao-xian chao-xian requested a review from alex-ju July 8, 2021 14:59
Copy link
Contributor

@alex-ju alex-ju left a comment

Choose a reason for hiding this comment

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

🆎

@chao-xian chao-xian merged commit 900ab57 into main Jul 15, 2021
@chao-xian chao-xian deleted the ab-test-header branch July 15, 2021 18:01
chao-xian added a commit that referenced this pull request Oct 18, 2021
The test has concluded and all users now get the new header.

Undoes:

- #6164
- #6225
chao-xian added a commit that referenced this pull request Oct 18, 2021
The test has concluded and all users now get the new header.

Undoes:

- #6164
- #6225
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.

2 participants