Skip to content

⏳ Feat: Report errors to Sentry / Configuration#778

Open
shmeleva wants to merge 2 commits intofeat/send-errors-to-sentryfrom
feat/send-errors-to-sentry-configuration
Open

⏳ Feat: Report errors to Sentry / Configuration#778
shmeleva wants to merge 2 commits intofeat/send-errors-to-sentryfrom
feat/send-errors-to-sentry-configuration

Conversation

@shmeleva
Copy link
Contributor

@shmeleva shmeleva commented Jun 1, 2025

WHY:

Continues #776.

What has been changed (if possible, add screenshots, gifs, etc. )

  • Add staticSettings.sentry.enabled.
  • Define handleError based on the value of staticSettings.sentry.enabled — use default handler if Sentry is disabled.
  • Do not initialise Sentry if Sentry is disabled.
  • Cannot dynamically import from Sentry in hooks.*.ts — these do not support top-level await:s.
  • Add default env variables that Sentry uses to upload source maps. Note: Source maps do not get uploaded if NODE_ENV=development. Change const config to async function to support dynamic import.

Check off each of the following tasks as they are completed

  • I have reviewed the changes myself in this PR. Please check the self-review document
  • I have added or edited unit tests.
  • I have run the unit tests successfully.
  • I have run the e2e tests successfully.
  • I have tested this change on my own device.
  • I have tested this change on other devices (Using Browserstack is recommended).
  • I have tested my changes using the WAVE extension
  • I have tested my changes using keyboard navigation and screen-reading
  • I have added documentation where necessary.
  • Is there an existing issue linked to this PR?
  • I have cleaned up the commit history and checked the commits follow the guidelines

@shmeleva shmeleva mentioned this pull request Jun 1, 2025
11 tasks
@shmeleva shmeleva marked this pull request as ready for review June 14, 2025 10:57
@kaljarv kaljarv changed the title Feat: Report errors to Sentry / Configuration ⏳ Feat: Report errors to Sentry / Configuration Aug 23, 2025
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.

1 participant