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 definition #26646

Closed
wants to merge 5 commits into from
Closed

Add AB Test definition #26646

wants to merge 5 commits into from

Conversation

lindseydew
Copy link
Contributor

@lindseydew lindseydew commented Oct 16, 2023

What is the value of this and can you measure success?

This adds an AB Test to test whether showing the sign in gate on every article in the morning increases sign in as per this hypothesis doc: https://docs.google.com/document/d/1c-xv7By5U8bvvsifglOetzrf9woDWncINPhTPwLKxlA/edit

What does this change?

Adds AB test switch

Notes

  • Need to confirm audience size and test expiry with D&I

DCR PR:

guardian/dotcom-rendering#9138

Screenshots

Checklist


export const signInGateTimesOfDay = {
id: 'SignInGateTimesOfDay',
start: '2020-05-20',
Copy link
Contributor Author

Choose a reason for hiding this comment

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

To do: Confirm on start and end date

author: 'Lindsey Dew',
description:
'Show sign in gate more frequently to users in the morning',
audience: 0.4,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

To do: confirm on audience segmentation

@lindseydew lindseydew marked this pull request as ready for review October 23, 2023 10:40
@lindseydew lindseydew requested a review from a team as a code owner October 23, 2023 10:40
description:
'Show sign in gate more frequently to users in the morning',
audience: 0.5,
audienceOffset: 0.4,
Copy link
Member

Choose a reason for hiding this comment

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

I think the main gate AB definition needs to be amended to match in frontend too.

Copy link
Contributor

"This PR is stale because it has been open 30 days with no activity. Unless a comment is added or the “stale” label removed, this will be closed in 3 days"

@github-actions github-actions bot added the Stale label Nov 23, 2023
Copy link
Contributor

This PR was closed because it has been stalled for 3 days with no activity.

@github-actions github-actions bot closed this Nov 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants