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

Set up Sentry to capture and output front end error messages #252

Merged
merged 10 commits into from
Nov 4, 2022
Merged

Set up Sentry to capture and output front end error messages #252

merged 10 commits into from
Nov 4, 2022

Conversation

create-issue-branch[bot]
Copy link
Contributor

@create-issue-branch create-issue-branch bot commented Oct 27, 2022 •
edited by loiswells97

What's Changed?

  • Added sentry reporting to log errors from the app, web component and those generated by running the user's code
  • Added environments for review apps (review), staging and production

Note:

Due to the different instances of Sentry initialised in the app and web component, errors from these sources will be reported separately

closes #122

@create-issue-branch create-issue-branch bot temporarily deployed to previews/issues/122-Set_up_Sentry_to_capture_and_output_front_end_error_messages October 27, 2022 13:22 Inactive
@github-actions
Copy link

@github-actions
Copy link

@loiswells97 loiswells97 temporarily deployed to previews/issues/122-Set_up_Sentry_to_capture_and_output_front_end_error_messages October 27, 2022 14:08 Inactive
@github-actions
Copy link

@loiswells97 loiswells97 temporarily deployed to previews/issues/122-Set_up_Sentry_to_capture_and_output_front_end_error_messages October 27, 2022 14:16 Inactive
@github-actions
Copy link

@loiswells97 loiswells97 temporarily deployed to previews/issues/122-Set_up_Sentry_to_capture_and_output_front_end_error_messages October 27, 2022 14:28 Inactive
@github-actions
Copy link

glenpike
glenpike previously approved these changes Oct 27, 2022
View reviewed changes
Copy link

@glenpike glenpike left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@loiswells97 loiswells97 temporarily deployed to previews/issues/122-Set_up_Sentry_to_capture_and_output_front_end_error_messages October 31, 2022 17:12 Inactive
@github-actions
Copy link

@loiswells97 loiswells97 temporarily deployed to previews/issues/122-Set_up_Sentry_to_capture_and_output_front_end_error_messages October 31, 2022 17:20 Inactive
@github-actions
Copy link

@loiswells97 loiswells97 marked this pull request as ready for review October 31, 2022 17:31
@loiswells97 loiswells97 temporarily deployed to previews/issues/122-Set_up_Sentry_to_capture_and_output_front_end_error_messages November 1, 2022 14:37 Inactive
@github-actions
Copy link

github-actions bot commented Nov 1, 2022

@github-actions
Copy link

github-actions bot commented Nov 1, 2022

@loiswells97 loiswells97 temporarily deployed to previews/issues/122-Set_up_Sentry_to_capture_and_output_front_end_error_messages November 1, 2022 17:30 Inactive
@github-actions
Copy link

github-actions bot commented Nov 1, 2022

@github-actions
Copy link

github-actions bot commented Nov 4, 2022

@sHtev sHtev merged commit 230277f into main Nov 4, 2022
@sHtev sHtev deleted the issues/122-Set_up_Sentry_to_capture_and_output_front_end_error_messages branch November 4, 2022 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sHtev sHtev sHtev approved these changes

@glenpike glenpike glenpike left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Set up Sentry to capture and output front end error messages
3 participants
@glenpike @sHtev @loiswells97