Skip to content

Depend on web-vitals 2.x - #6

Open
laradevitt wants to merge 1 commit into
bejamas:masterfrom
laradevitt:update_web_vitals_lib
Open

Depend on web-vitals 2.x#6
laradevitt wants to merge 1 commit into
bejamas:masterfrom
laradevitt:update_web_vitals_lib

Conversation

@laradevitt

Copy link
Copy Markdown

This is a PR for Update web-vitals dependency to 2.x branch #5

With this update there is a breaking change as described in the PR Update CLS to max session window 5s cap 1s gap #148:

The only observable difference developers might notice to the API is previously the metric.entries array would only ever add new entries as additional shifts occurred on the page. With this change, only the entries from the max session window are reported, which means it could be the case the some of the entries referenced when CLS is first reported (e.g. after the tab is backgrounded), may not be the entries that are referenced the next time the tab is backgrounded. Developers not referencing the metric.entries array should not notice any differences other than a possible reduction in the value reported.

AFAICT this plugin only references the performance entry of the TTFB metric to calculate request time so I'm fairly confident this plugin would be unaffected by the breaking change but a proper review is obviously in order.

Thanks!

@laradevitt

Copy link
Copy Markdown
Author

Wups - Would actually be updating from web-vitals ^0.2.1 (not 1.x) so it might take a bit more work to bring this plugin up-to-date.

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

Successfully merging this pull request may close these issues.

1 participant