-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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
refactor: lazy-load sentry #6128
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Lighthouse Results
|
this thought crossed my mind too 👍 |
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.
all for iterating on what's best
cc @nodejs/web-infra I'm fast-tracking this for now. We might want to revisit the behaviour of Sentry Tracing as we have a meeting with Sentry next week. |
Description
Sentry has an obnoxious big bundle-size which is non-tree-shakeable (mostly) due to its tracing/replay features.
After pondering a bit I've concluded the following:
vercel/analytics
)Validation
We should still see Sentry reporting Errors on client-side and it should still do
/monitoring
API calls