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

NewBrowse AB test list view #3614

Merged
merged 4 commits into from
May 14, 2024
Merged

NewBrowse AB test list view #3614

merged 4 commits into from
May 14, 2024

Conversation

hannako
Copy link
Contributor

@hannako hannako commented May 13, 2024

What

Adds a new partial for users allocated a variant B cookie.

Related PR in govuk-fastly alphagov/govuk-fastly#76 (review)

Variant A or Z Variant B
Screenshot 2024-05-13 at 17 02 46 Screenshot 2024-05-13 at 17 03 42

How to view the B variant before we turn the AB test on

To toggle between the variants install modheader and then add a new Request Header by clicking the +. Enter a Name of govuk-abtest-newbrowse and a Value of B

Trello: https://trello.com/c/VUPBYG8Y/2594-ramp-up-newbrowse-ab-test-5-to-b-for-10-days

hannako and others added 3 commits May 13, 2024 16:37
Adds chevron-card from `frontend`, the view template and styles have been changed slightly to ensure it still works as expecting when used in collections. the main differences are:

- Allow custom heading levels to be used
- `data_attributes` now passed in from the parent view template, this matches the current behaviour
- Ensure card content has a max-width of two thirds on desktop devices
@govuk-ci govuk-ci temporarily deployed to collections-pr-3614 May 13, 2024 15:57 Inactive
Co-authored-by: Martin Jones <martin.jones@digital.cabinet-office.gov.uk>
@govuk-ci govuk-ci temporarily deployed to collections-pr-3614 May 13, 2024 15:59 Inactive
@hannako hannako marked this pull request as ready for review May 13, 2024 16:04
Copy link
Contributor

@beccapearce beccapearce left a comment

Choose a reason for hiding this comment

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

Looks good!

@hannako hannako merged commit bcc3eed into main May 14, 2024
18 checks passed
@hannako hannako deleted the start_newbrowse_test branch May 14, 2024 09:16
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.

None yet

4 participants