Skip to content

Commit

Permalink
Merge branch 'main' into ei/move-initPermutive
Browse files Browse the repository at this point in the history
  • Loading branch information
emma-imber authored Feb 5, 2025
2 parents ce230ed + cac500b commit edf7ad3
Show file tree
Hide file tree
Showing 5 changed files with 6 additions and 37 deletions.
5 changes: 5 additions & 0 deletions .changeset/olive-bees-think.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'@guardian/commercial': minor
---

Remove 0% test for The Trade Desk and set live
2 changes: 0 additions & 2 deletions src/experiments/ab-tests.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@ import type { ABTest } from '@guardian/ab-core';
import { mpuWhenNoEpic } from './tests/mpu-when-no-epic';
import { optOutFrequencyCap } from './tests/opt-out-frequency-cap';
import { prebidKeywords } from './tests/prebid-keywords';
import { ttdPrebidBidder } from './tests/ttd-prebid';

/**
* You only need to add tests to this file if the code you are testing is here in
Expand All @@ -14,5 +13,4 @@ export const concurrentTests: ABTest[] = [
mpuWhenNoEpic,
optOutFrequencyCap,
prebidKeywords,
ttdPrebidBidder,
];
28 changes: 0 additions & 28 deletions src/experiments/tests/ttd-prebid.ts

This file was deleted.

3 changes: 1 addition & 2 deletions src/lib/header-bidding/prebid/bid-config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,6 @@ import {
shouldIncludeKargo,
shouldIncludeMagnite,
shouldIncludeOpenx,
shouldIncludeTheTradeDesk,
shouldIncludeTripleLift,
shouldIncludeTrustX,
shouldIncludeXaxis,
Expand Down Expand Up @@ -684,7 +683,7 @@ const currentBidders = (
[shouldIncludeOpenx(), openxClientSideBidder(pageTargeting)],
[shouldIncludeKargo(), kargoBidder],
[shouldIncludeMagnite(), magniteBidder],
[shouldIncludeTheTradeDesk(), theTradeDeskBidder(gpid)],
[true, theTradeDeskBidder(gpid)],
];

const otherBidders = biddersToCheck
Expand Down
5 changes: 0 additions & 5 deletions src/lib/header-bidding/utils.ts
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
import { isString } from '@guardian/libs';
import { once } from 'lodash-es';
import { isUserInVariant } from '../../experiments/ab';
import { ttdPrebidBidder } from '../../experiments/tests/ttd-prebid';
import { createAdSize } from '../../lib/ad-sizes';
import {
isInAuOrNz,
Expand Down Expand Up @@ -191,9 +189,6 @@ export const shouldIncludeKargo = (): boolean => isInUsa();
export const shouldIncludeMagnite = (): boolean =>
!!window.guardian.config.switches.prebidMagnite;

export const shouldIncludeTheTradeDesk = (): boolean =>
isUserInVariant(ttdPrebidBidder, 'variant');

export const shouldIncludeMobileSticky = once(
(): boolean =>
window.location.hash.includes('#mobile-sticky') ||
Expand Down

0 comments on commit edf7ad3

Please sign in to comment.