-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Extract TTFB metric to getLoadingDurations util #47541
Conversation
Size Change: 0 B Total Size: 1.31 MB ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
1d2cb9e
to
5d417ee
Compare
I'm a bit confused about the existing error:
There's no |
5d417ee
to
f0bc52f
Compare
Hmm, I think this error is referring to the Reading this, perhaps we should be wrapping |
I believe this is because we run Maybe it's time to refactor this util to make it more resilient. Using |
Closing this for now, it doesn't seem the right approach. |
What?
Refactors the TimeToFirstByte performance metric to the
getLoadingDurations
utility.Why?
This makes it work like any other metric.
How?
By moving the calculation to the
getLoadingDurations
utility and use it directly in the test.Testing Instructions
Go to the performance CI job of this PR and verify that the front-end metrics are still reported and work as expected.