Skip to content
This repository has been archived by the owner on Sep 1, 2022. It is now read-only.

feat!: Load web-vitals asynchronously #322

Merged
merged 2 commits into from
Mar 29, 2022
Merged

Conversation

mxdvl
Copy link
Contributor

@mxdvl mxdvl commented Mar 22, 2022

What does this change?

Prevents loading the web-vitals module when the user is not in the sampling group and the sampling has not been bypassed.

Why?

Less JS sent to the vast majority (~99%) of our users.

BREAKING CHANGE: async initCoreWebVitals & bypassCoreWebVitalsSampling

This prevents loading the web-vitals module when the user is not in
the sampling group and the sampling has not been bypassed.

Less JS sent to the vast majority (~99%) of our users.
@mxdvl mxdvl added the enhancement New feature or request label Mar 22, 2022
@mxdvl mxdvl force-pushed the mxdvl/load-web-vitals-async branch from fa648c0 to e343d14 Compare March 22, 2022 12:14
@coveralls
Copy link

Coverage Status

Coverage increased (+0.0004%) to 99.81% when pulling e343d14 on mxdvl/load-web-vitals-async into add9281 on main.

Copy link
Contributor

@SiAdcock SiAdcock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great change 👍

@mxdvl mxdvl merged commit feb8346 into main Mar 29, 2022
@mxdvl mxdvl deleted the mxdvl/load-web-vitals-async branch March 29, 2022 12:33
@github-actions
Copy link

🎉 This PR is included in version 4.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants