Closed NicolasMassart closed 4 days ago
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code
Description
Convert
Logger.message
toLogger.log
as to provide breadcrumb for next event instead of filling Sentry with events that are hitting our quotas.Our
Logger.message
util internally uses SentrycaptureMessage
andLogger.message
util internally uses SentryaddBreadcrumb
.Sentry breadcrumbs vs message
Improvement suggestion
In order to prevent this
Logger.message
to be used again I recommend to completely remove it from utils and force the use of either logs or errors. We had only 3 places it was used, as they are now removed, deleting the utility function has no impact.Related issues
see MetaMask/mobile-planning/issues/1814
Manual testing steps
NA
Screenshots/Recordings
Before
NA
After
NA
Pre-merge author checklist
Pre-merge reviewer checklist