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
feat(tracing): Promote enableLongTask
to option of BrowserTracing
#6837
Conversation
@@ -124,7 +134,8 @@ const DEFAULT_BROWSER_TRACING_OPTIONS: BrowserTracingOptions = { | |||
routingInstrumentation: instrumentRoutingWithDefaults, | |||
startTransactionOnLocationChange: true, | |||
startTransactionOnPageLoad: true, | |||
_experiments: { enableLongTask: true, enableInteractions: false }, | |||
enableLongTask: true, | |||
_experiments: {}, |
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.
m: Did we want to remove enableInteractions
?
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.
That is false by default, so should be the same as not having it at all, I figured?
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.
yeah good point, save some bytes!
size-limit report 📦
|
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.
We'll need to add the option to docs as well then.
Good point! I created a PR that we can merge once this is released: getsentry/sentry-docs#6104 |
This experiment was on by default, which showed that this is not actually an experiment anymore, and thus should be promoted to a regular option.