Skip to content

Commit

Permalink
Merge pull request #13079 from guardian/doml/remove-big-six-ab-test
Browse files Browse the repository at this point in the history
Remove Onwards Content (big six) AB test
  • Loading branch information
domlander authored Jan 7, 2025
2 parents 1994408 + b1be3f3 commit 0250dc3
Show file tree
Hide file tree
Showing 4 changed files with 1 addition and 275 deletions.
222 changes: 0 additions & 222 deletions dotcom-rendering/src/components/BigSixOnwardsContent.tsx

This file was deleted.

22 changes: 1 addition & 21 deletions dotcom-rendering/src/components/OnwardsUpper.importable.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -6,12 +6,10 @@ import {
Pillar,
} from '../lib/articleFormat';
import type { EditionId } from '../lib/edition';
import { useAB } from '../lib/useAB';
import { useIsAndroid } from '../lib/useIsAndroid';
import { palette } from '../palette';
import type { OnwardsSource } from '../types/onwards';
import type { TagType } from '../types/tag';
import { BigSixOnwardsContent } from './BigSixOnwardsContent';
import { FetchOnwardsData } from './FetchOnwardsData.importable';
import { Section } from './Section';

Expand Down Expand Up @@ -216,14 +214,7 @@ export const OnwardsUpper = ({
discussionApiUrl,
absoluteServerTimes,
}: Props) => {
const abTestAPI = useAB()?.api;
const isInOnwardsAbTestVariant = abTestAPI?.isUserInVariant(
'OnwardsContentArticle',
'variant',
);

const isAndroid = useIsAndroid();

if (isAndroid) return null;

// Related content can be a collection of articles based on
Expand Down Expand Up @@ -306,18 +297,7 @@ export const OnwardsUpper = ({

return (
<div css={onwardsWrapper}>
{!!url && isInOnwardsAbTestVariant && (
<Section
fullWidth={true}
borderColour={palette('--article-border')}
>
<BigSixOnwardsContent
url={url}
discussionApiUrl={discussionApiUrl}
/>
</Section>
)}
{!!url && !isInOnwardsAbTestVariant && (
{!!url && (
<Section
fullWidth={true}
borderColour={palette('--article-border')}
Expand Down
2 changes: 0 additions & 2 deletions dotcom-rendering/src/experiments/ab-tests.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ import { adBlockAsk } from './tests/ad-block-ask';
import { consentlessAds } from './tests/consentless-ads';
import { integrateIma } from './tests/integrate-ima';
import { mpuWhenNoEpic } from './tests/mpu-when-no-epic';
import { onwardsContentArticle } from './tests/onwards-content-article';
import { optimiseSpacefinderInline } from './tests/optimise-spacefinder-inline';
import { signInGateMainControl } from './tests/sign-in-gate-main-control';
import { signInGateMainVariant } from './tests/sign-in-gate-main-variant';
Expand All @@ -20,5 +19,4 @@ export const tests: ABTest[] = [
mpuWhenNoEpic,
adBlockAsk,
optimiseSpacefinderInline,
onwardsContentArticle,
];
30 changes: 0 additions & 30 deletions dotcom-rendering/src/experiments/tests/onwards-content-article.ts

This file was deleted.

0 comments on commit 0250dc3

Please sign in to comment.