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

Capture console.error in dApp with Sentry #875

Merged
merged 1 commit into from
Nov 18, 2024

Conversation

nkuba
Copy link
Member

@nkuba nkuba commented Nov 18, 2024

The dApp sometime catches errors and logs them to console without bubbling it up. This makes Sentry miss some errors. This PR captures console.error and sends it to Sentry using the CaptureConsole plugin: https://docs.sentry.io/platforms/javascript/configuration/integrations/captureconsole/

The dApp sometime catches errors and logs them to console without
bubbling it up. This makes Sentry miss some errors. This PR captures
console.error and sends it to Sentry using the `CaptureConsole` plugin:
https://docs.sentry.io/platforms/javascript/configuration/integrations/captureconsole/
@nkuba nkuba requested a review from a team November 18, 2024 10:28
@nkuba nkuba self-assigned this Nov 18, 2024
@nkuba nkuba added this to the Ledger Live Launch milestone Nov 18, 2024
Copy link

netlify bot commented Nov 18, 2024

Deploy Preview for acre-dapp-testnet ready!

Name Link
🔨 Latest commit 157017c
🔍 Latest deploy log https://app.netlify.com/sites/acre-dapp-testnet/deploys/673b16bf1e8ea90008c9afed
😎 Deploy Preview https://deploy-preview-875--acre-dapp-testnet.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Nov 18, 2024

Deploy Preview for acre-dapp ready!

Name Link
🔨 Latest commit 157017c
🔍 Latest deploy log https://app.netlify.com/sites/acre-dapp/deploys/673b16bfb1ff1c000867f9ed
😎 Deploy Preview https://deploy-preview-875--acre-dapp.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@michalinacienciala michalinacienciala merged commit 6fb6b56 into main Nov 18, 2024
28 checks passed
@michalinacienciala michalinacienciala deleted the dapp-capture-console-log branch November 18, 2024 10:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants