search

Financial-Times / n-service-worker

❌ DECOMMISSIONED Global service worker component for next.ft.com
5 stars 2 forks source link

Deleted ads cache 🐿 v2.12.5 #169

Closed alexflorisca closed 2 years ago

alexflorisca commented 4 years ago

We decided to deleted the ads cache from the SW to simplify things. It was an experiment to see if caching third party scripts loaded by the o-ads library will improve performance. The results show that there was no change in performance and it's a hangover from a few years ago when this was carried out.

We also switched to using Google Tag Manager to keep track of third party scripts so it doesn't make sense to update the SW anytime one of these scripts changes.

alexflorisca commented 2 years ago

Tidying up my open PRs, I'm closing this, feel free to re-open if still needed :)