Skip to content

Ignore Picture-in-Picture Sentry noise#255

Closed
kentcdodds wants to merge 2 commits into
mainfrom
ignore-picture-in-picture-sentry-noise
Closed

Ignore Picture-in-Picture Sentry noise#255
kentcdodds wants to merge 2 commits into
mainfrom
ignore-picture-in-picture-sentry-noise

Conversation

@kentcdodds
Copy link
Copy Markdown
Member

@kentcdodds kentcdodds commented May 17, 2026

Summary

  • Add an epicshop postinstall patch for the workshop app client Sentry beforeSend hook
  • Drop only browser DOMExceptions where type is NotAllowedError and the value is exactly The video element is processing a Picture-in-Picture request.
  • Document why the patch must remain narrow so unrelated NotAllowedError exceptions still report
  • Update the epicshop lockfile metadata for the new install script

Validation

  • npm --prefix epicshop run postinstall
  • npm run typecheck
  • npm run lint
Open in Web Open in Cursor 

cursoragent and others added 2 commits May 17, 2026 14:15
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
@kody-bot kody-bot closed this May 17, 2026
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.

3 participants