Merge pull request #1097 from Financial-Times/ADSDEV-2104-moat-deprec…

…ation-remove-moat-ads-traffic-flag-and-ads-moat-timeout-from-the-codebase ADSDEV-2104: Removing moatAdsTraffic flag
Financial-Times · Nov 27, 2024 · cf60250 · cf60250
2 parents 7ebf08c + e669f43
commit cf60250
Show file tree

Hide file tree

Showing 4 changed files with 3,147 additions and 6,420 deletions.
diff --git a/examples/kitchen-sink/client/main.js b/examples/kitchen-sink/client/main.js
@@ -35,12 +35,7 @@ readyState.domready.then(async () => {
       lazyLoadMargins: {
         760: '15%',
         980: '5%'
-      },
-      waitForMoat: true
-    }
-
-    if (flagsClient.get('moatAdsTraffic')) {
-      displayAds.validateTraffic?.()
+      }
     }
 
     // Fetch the ads data

diff --git a/examples/kitchen-sink/package.json b/examples/kitchen-sink/package.json
@@ -11,7 +11,7 @@
     "test": "../../node_modules/.bin/jest"
   },
   "dependencies": {
-    "@financial-times/ads-display": "^5.2.8",
+    "@financial-times/ads-display": "^6.3.0",
     "@financial-times/dotcom-middleware-app-context": "file:../../packages/dotcom-middleware-app-context",
     "@financial-times/dotcom-middleware-asset-loader": "file:../../packages/dotcom-middleware-asset-loader",
     "@financial-times/dotcom-middleware-navigation": "file:../../packages/dotcom-middleware-navigation",

diff --git a/examples/kitchen-sink/server/controllers/home.js b/examples/kitchen-sink/server/controllers/home.js
@@ -23,7 +23,6 @@ const flagsStore = {
   tracking: true,
   oTracking: true,
   AdsPermutive: true,
-  moatAdsTraffic: true,
   adsEnableTestCreatives: false
 }