search

Financial-Times / n-ui

❌ DEPRECATED Server, build and client side bootstrapping for FT.com's user-facing applications.
7 stars 5 forks source link

Remove the adsDisableMetricsSampling flag 🐿 v2.12.5 #1530

Closed alexflorisca closed 4 years ago

alexflorisca commented 4 years ago

Removing the adsDisableMetricsSampling flag and making the feature permanent. No user facing changes should be visible

SamiTriki commented 4 years ago

The change looks good, do you think it's worth looking at the setupAdMetrics method since we don't rely on a flag anymore and that the method still had a flag as an argument @carlesandres