-
Notifications
You must be signed in to change notification settings - Fork 105
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:[IOPLT-978] Changes the Sentry integration for performance monitoring #6741
Conversation
Jira Pull Request LinkThis Pull Request refers to Jira issues: |
PR Title Validation for conventional commit type✅ All good! PR title follows the conventional commit type. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #6741 +/- ##
=======================================
Coverage 50.30% 50.30%
=======================================
Files 1563 1563
Lines 32510 32510
Branches 7350 7349 -1
=======================================
Hits 16353 16353
+ Misses 16117 16105 -12
- Partials 40 52 +12
... and 12 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
@shadowsheep1 Fixed all the references |
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.
LGTM
Short description
This PR changes the integration of Sentry SDK to remove navigation instrumentation and handle the base ReactNative one.
How to test
Check Sentry events