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

chore: Remove navigation instrumentation #12174

Merged
merged 22 commits into from
Nov 11, 2024
Merged
Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
5e2fe73
Remove react navigation instrumentation from sentry configuration
tommasini Nov 4, 2024
57c9c34
Removing remaining of sentry react native instrumentation, this way c…
tommasini Nov 7, 2024
5a3b707
Load script custom trace
tommasini Nov 7, 2024
7573a8b
custom instrumentation spans for load bundle, login and network selec…
tommasini Nov 8, 2024
bd89a86
Merge branch 'main' into fix/remove-sentry-navigation-instrumentation
tommasini Nov 8, 2024
bdc7f74
Cleanup code re using variable and added better tracing to login flow
tommasini Nov 8, 2024
344297c
add tags and account watch
tommasini Nov 8, 2024
5c3b3c9
update Performance file
tommasini Nov 8, 2024
70642a6
fix typescript at Performance file
tommasini Nov 8, 2024
847effe
fix unit test at login screen
tommasini Nov 8, 2024
ecea9d7
remove unnecessary log
tommasini Nov 8, 2024
68d6962
Merge branch 'main' into fix/remove-sentry-navigation-instrumentation
tommasini Nov 8, 2024
2d76dc6
fix unit tests
tommasini Nov 8, 2024
312474b
remove unused imports
tommasini Nov 8, 2024
78363b2
UIStartup span test file
tommasini Nov 8, 2024
bc4b0ab
space to UI Startup span
tommasini Nov 8, 2024
0a2c95b
getUiStartupSpan test file
tommasini Nov 8, 2024
24aa062
feat: added Engine Trace and OP names to traces
MarioAslau Nov 11, 2024
d6f4009
Merge branch 'main' into fix/remove-sentry-navigation-instrumentation
MarioAslau Nov 11, 2024
b841b3d
fix unit tests on uistartup
tommasini Nov 11, 2024
fc207fb
fix trace name ui startup at sentry utils
tommasini Nov 11, 2024
26ceb41
merge main and solve conflicts
tommasini Nov 11, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 2 additions & 9 deletions app/util/sentry/utils.js
Original file line number Diff line number Diff line change
Expand Up @@ -496,20 +496,13 @@ export function setupSentry() {
dsn,
debug: isDev,
environment,
integrations:
metricsOptIn === AGREED
? [
...integrations,
new Sentry.reactNativeTracingIntegration({
routingInstrumentation,
}),
]
: integrations,
integrations,
// Set tracesSampleRate to 1.0, as that ensures that every transaction will be sent to Sentry for development builds.
tracesSampleRate: isDev || isQa ? 1.0 : 0.04,
beforeSend: (report) => rewriteReport(report),
beforeBreadcrumb: (breadcrumb) => rewriteBreadcrumb(breadcrumb),
beforeSendTransaction: (event) => excludeEvents(event),
enabled: metricsOptIn === AGREED,
});
};
init();
Expand Down
Loading