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.

Sign up for GitHub

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

Jump to bottom

🤖 348 adding sentry to the web app #394

Merged
merged 3 commits into from
May 4, 2022
Merged

🤖 348 adding sentry to the web app #394

merged 3 commits into from
May 4, 2022

Conversation

xoscar
Copy link
Collaborator

@xoscar xoscar commented May 2, 2022

This PR adds sentry to catch any front end application errors.

Changes

  • Adds error boundary and sentry config.

Checklist

  • tested locally
  • added new dependencies
  • updated the docs
  • added a test

@xoscar xoscar requested a review from mortada-codes May 2, 2022 19:21
@xoscar xoscar self-assigned this May 2, 2022
@xoscar xoscar linked an issue May 2, 2022 that may be closed by this pull request
Integrate sentry to the demo #348
Closed
@xoscar xoscar requested review from cescoferraro and removed request for mortada-codes May 2, 2022 19:21
@xoscar xoscar added the frontend label May 2, 2022
@xoscar xoscar merged commit db44986 into main May 4, 2022
@xoscar xoscar deleted the 348-integrate-sentry-to-the-demo branch May 4, 2022 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cescoferraro cescoferraro Awaiting requested review from cescoferraro

Assignees

@xoscar xoscar

Labels
frontend
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Integrate sentry to the demo
1 participant
@xoscar