Skip to content
This repository has been archived by the owner on Mar 14, 2024. It is now read-only.

Add bfcache not restored reasons api #10338

Merged
merged 2 commits into from
Aug 15, 2023

Conversation

tunetheweb
Copy link
Member

@tunetheweb tunetheweb commented Aug 14, 2023

Changes proposed in this pull request:

  • Add no restored reasons API analytics as noticed we have a lot less bfcache usage on desktop than mobile and would like to understand why
  • Also found a small bug when attribution.eventEntry is not set so included that fix too.

Equivalent change to d.c.c. is: GoogleChrome/developer.chrome.com#7074

@netlify
Copy link

netlify bot commented Aug 14, 2023

Deploy Preview for web-dev-staging ready!

Name Link
🔨 Latest commit 67bfb1d
🔍 Latest deploy log https://app.netlify.com/sites/web-dev-staging/deploys/64daa529b6f9a30008529e7b
😎 Deploy Preview https://deploy-preview-10338--web-dev-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@matthiasrohmer matthiasrohmer merged commit a2c6146 into main Aug 15, 2023
10 checks passed
@matthiasrohmer matthiasrohmer deleted the add-bfcache-not-restored-reasons-api branch August 15, 2023 08:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants