search

nuxt-community / sentry-module

Sentry module for Nuxt 2
https://sentry.nuxtjs.org
MIT License
500 stars 113 forks source link

fix: only include 'crash' type for `ReportingObserver` integration #560

Closed rchl closed 1 year ago

rchl commented 1 year ago

I'm not sure why we even enable ReportingObserver integration by default since it seems to produce a lot of reports that are not really that useful. But instead of removing it completely, only include the crash report types (so exclude deprecation and intervention by default) to keep those reports to minimum.

It could be considered a breaking change but from my experience those reports are just noise so I'm handling it more like a bugfix for default options.

github-actions[bot] commented 1 year ago

size-limit report 📦

Path Size
fixture: base 364.9 KB (+0.01% 🔺)
fixture: lazy 369.3 KB (+0.01% 🔺)
fixture: tracing 381.48 KB (+0.01% 🔺)
fixture: lazy+tracing 385.85 KB (+0.01% 🔺)