Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cannot read properties of undefined (reading 'requestStart') on navigation with bfcache #220

Closed
meskill opened this issue Feb 3, 2023 · 12 comments

Comments

@meskill
Copy link

meskill commented Feb 3, 2023

I'm getting the error from the title on this line when testing bfcache in chrome browser.

Reproduction

  1. Open Chrome DevTools -> Application -> Back/Forward Cache
  2. Click Test Back/Forward cache

image

@SuperOleg39
Copy link
Contributor

Initial problem from web-vitals package - GoogleChrome/web-vitals#313

@SuperOleg39
Copy link
Contributor

Following the discussion in web-vitals repo, we can still report 0 values and add just fix for BFCache case - #221

@scamden
Copy link

scamden commented Mar 22, 2024

i believe we are hitting this error as well fwiw

undefined is not an object (evaluating 'e.entries[0].requestStart')

@Zizzamia
Copy link
Owner

@scamden what version are you using?

@scamden
Copy link

scamden commented Mar 25, 2024

8.4.0, should i try an upgrade?

@scamden
Copy link

scamden commented Mar 26, 2024

I'm upgrading to 9.4 and will let you know if we see again

@Zizzamia
Copy link
Owner

Also, curios, what version of web-vitals are you resolving, maybe we need to upgrade from Perfume side as well.

@scamden
Copy link

scamden commented Mar 26, 2024

Also, curios, what version of web-vitals are you resolving, maybe we need to upgrade from Perfume side as well.

@Zizzamia we have 3.5.2 for web vitals

@scamden
Copy link

scamden commented Mar 26, 2024

prior to the upgrade 3.4.0 fwiw

@Zizzamia
Copy link
Owner

@scamden do you still see the issue after upgrade?

@scamden
Copy link

scamden commented Apr 11, 2024

Haven't seen it since to my knowledge (although we had only seen it once so sample size is pretty low for this data :P )

Thanks for the tip on upgrading! Perfume is a great package :)

@Zizzamia
Copy link
Owner

Ok, so I think it's safe to close this for now.

I will reopen this if this issue happens again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants