Skip to content

Commit

Permalink
Merge pull request #12597 from guardian/doml/start-us-banner-test
Browse files Browse the repository at this point in the history
Set US Banner AB test audience to 15%
  • Loading branch information
domlander authored Oct 21, 2024
2 parents 5c44991 + a861511 commit 5b4f016
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ import { ExpandableMarketingCard } from './ExpandableMarketingCard';
import { ExpandableMarketingCardSwipeable } from './ExpandableMarketingCardSwipeable';
import { Hide } from './Hide';

const isFirstArticle = () => {
const isFirstOrSecondArticle = () => {
const [dailyCount = {} as DailyArticle] = getDailyArticleCount() ?? [];
return Object.keys(dailyCount).length === 0 || dailyCount.count <= 1;
};
Expand All @@ -25,7 +25,7 @@ const isNewUSUser = async () => {
!!editionCookie && editionCookie !== 'US';

// This check must happen AFTER we've ensured that the user is in the US.
const isNewUser = isFirstArticle();
const isNewUser = isFirstOrSecondArticle();

return !hasUserSelectedNonUSEdition && isNewUser;
};
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ export const UsaExpandableMarketingCard: ABTest = {
author: '[email protected]',
description:
'Test the impact of showing the user a component that highlights the Guardians journalism.',
audience: 0 / 100,
audience: 15 / 100,
audienceOffset: 0 / 100,
audienceCriteria: 'US-based users that see the US edition.',
successMeasure: 'Users are more likely to engage with the site.',
Expand Down

0 comments on commit 5b4f016

Please sign in to comment.