Skip to content

Add ignoreErrors to Sentry - #1564

Open
cocomarine wants to merge 4 commits into
mainfrom
1660-ignore-this-error-in-sentry
Open

Add ignoreErrors to Sentry#1564
cocomarine wants to merge 4 commits into
mainfrom
1660-ignore-this-error-in-sentry

Conversation

@cocomarine

@cocomarine cocomarine commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Part of https://github.com/RaspberryPiFoundation/digital-editor-issues/issues/1660
Screenshot 2026-07-30 at 13 34 19

Screenshot 2026-07-30 at 13 34 31
  • Initially thought it might have something to do with src/hooks/useContainerMinWidth.js which uses ResizeObserver within useLayoutEffect.
    • However, this error predates this file's creation.
    • Also, the stack trace was often useless/empty which seems to indicate it's not a real exception.
  • Turned out it's a native browser warning fired when nested ResizeObservers (CodeMirror, design-system-react's tooltip/dropdown positioning, the 3D canvas, resizable panels) react to each other's layout changes within one frame. So, not a real bug in our code.
  • Sentry's default GlobalHandlers integration picks it up and reports it as an error even though there's nothing actionable to fix.
  • So, Adding the errors (see screenshots) to ignoreErrors to suppress this frequent noise as was recommended by Sentry: https://sentry.io/answers/react-resizeobserver-loop-completed-with-undelivered-notifications/
  • The same filter also to be added to editor-standalone Sentry client.

@cocomarine
cocomarine temporarily deployed to previews/1564/merge July 30, 2026 12:39 — with GitHub Actions Inactive
@cocomarine
cocomarine temporarily deployed to previews/1564/merge July 30, 2026 12:54 — with GitHub Actions Inactive
@cocomarine
cocomarine marked this pull request as ready for review July 30, 2026 12:55
@cocomarine
cocomarine temporarily deployed to previews/1564/merge July 31, 2026 10:05 — with GitHub Actions Inactive
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