Skip to content
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(onboarding): Enable only errors ootb #86395

Merged
merged 2 commits into from
Mar 5, 2025

Conversation

matejminar
Copy link
Member

@matejminar matejminar commented Mar 5, 2025

This PR makes it so that only Error monitoring is instrumented in the onboarding snippets by default.
People can still opt-in to Tracing, Session Replay, and Profiling by explicitly checking the box.

The main problem we’re trying to solve is churn - based on the data it seems strongly correlated to usage of on-demand/PAYG and to using multiple products. We’ll make this change and re-visit in 45 days to see if it had the desired effect or not.

Before

CleanShot 2025-03-05 at 17 54 11

After

CleanShot 2025-03-05 at 17 53 49

In a follow-up PR, we'll:

  • fix the existing logic so that we don't have to skip some tests
  • omit the integrations: [] line in instructions if the array is empty

Trying to keep this PR as simple as possible as it's quite controversial.

@matejminar matejminar requested review from HazAT and a team March 5, 2025 16:58
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 5, 2025
@@ -196,13 +196,16 @@ export const platformProductAvailability = {
/**
* Defines which products are selected per default for each platform
* If not defined in here, all products are selected
*
* UPDATE Mar 2025, we're running an experiment that has only error monitoring enabled by default
Copy link
Member

Choose a reason for hiding this comment

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

This will age well 🙈

Copy link

codecov bot commented Mar 5, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #86395      +/-   ##
==========================================
- Coverage   87.83%   87.83%   -0.01%     
==========================================
  Files        9746     9746              
  Lines      552349   552348       -1     
  Branches    21533    21533              
==========================================
- Hits       485182   485181       -1     
  Misses      66778    66778              
  Partials      389      389              

@matejminar matejminar merged commit 0160de5 into master Mar 5, 2025
45 checks passed
@matejminar matejminar deleted the feat/enable-only-errors-in-onboarding branch March 5, 2025 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: Frontend Automatically applied to PRs that change frontend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants