getsentry / team-mobile

Meta issues for the Mobile team
4 stars 1 forks source link

ignoreErrors & ignoreTransactions Options #108

Open kahest opened 1 year ago

kahest commented 1 year ago

https://github.com/getsentry/develop/pull/884

Develop docs updated: https://develop.sentry.dev/sdk/features/#event-sampling

https://www.notion.so/sentry/RFC-for-ignoreTransactions-5ba68f17867e4bb08e80f1ec7f9805dd

### Tasks
- [ ] getsentry/sentry-cocoa#2917
- [ ] getsentry/sentry-java#2660
- [x] Android inherits from Java?
- [ ] getsentry/sentry-react-native#2974
- [ ] getsentry/sentry-dart#1391
- [ ] getsentry/sentry-dotnet#2306
marandaneto commented 1 year ago

Consider https://github.com/getsentry/develop/pull/884#issuecomment-1482524958

bitsandfoxes commented 1 year ago

Removing the label. Unity inherits this from .NET.

markushi commented 1 year ago

@smeubank regarding ignoreTransactions, I think there's still the open topic of client reports (how/if this should show up there) - do we have some decision on this already?

kahest commented 1 year ago

We will implement this on a case-by-case basis, no need to implement on all platforms right away - changed priority to P3 to reflect this.

kahest commented 1 year ago

About Client Reports: we aligned on this: