Remove adsDisableMetricsSampling flag 🐿 v2.12.5 (#1531)

Financial-Times · Jan 13, 2020 · af23728 · af23728
1 parent 0fd6d2e
commit af23728
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 4 deletions.
diff --git a/.pa11yci.js b/.pa11yci.js
@@ -2,7 +2,7 @@ const config = {
 	defaults: {
 		timeout: 10000,
 		headers: {
-			Cookie: 'next-flags=ads:off,sourcepoint:off;secure=true'
+			Cookie: 'next-flags=ads:off,sourcepoint:off,secure=true,enableGTM:off'
 		}
 	},
 	urls: [

diff --git a/components/n-ui/ads/index.js b/components/n-ui/ads/index.js
@@ -13,7 +13,7 @@ window.oAds = Ads;
 function initOAds (flags, appName, adOptions) {
 	const initObj = oAdsConfig(flags, appName, adOptions);
 
-	setupAdsMetrics(flags && flags.adsDisableMetricsSampling);
+	setupAdsMetrics(true);
 
 	utils.log('dfp_targeting', initObj.dfp_targeting);
 

diff --git a/components/n-ui/ads/test/index.spec.js b/components/n-ui/ads/test/index.spec.js
@@ -58,8 +58,7 @@ describe('Main', () => {
 
 	it('Should setup ads monitoring functionality', () => {
 		const flags = {
-			get: () => true,
-			adsDisableMetricsSampling: true
+			get: () => true
 		};
 		const setupMetricsStub = sandbox.stub(AdsMetrics, 'setupAdsMetrics');