Uh oh!
There was an error while loading. Please reload this page.
[v7] Remove eventbuilder exports which were historically used in @sentry/electron - #4887
Conversation
These exports were historically used in `@sentry/electron`, but are no longer being used by the Electron SDK or the React Native SDK, so they can be removed.
These exports were historically used in `@sentry/electron`, but are no longer being used by the Electron SDK or the React Native SDK, so they can be removed.
These exports were historically used in `@sentry/electron`, but are no longer being used by the Electron SDK or the React Native SDK, so they can be removed.
These exports were historically used in `@sentry/electron`, but are no longer being used by the Electron SDK or the React Native SDK, so they can be removed.
maclockard
commented
Sep 15, 2022
This is a breaking change that wasn't called out in the release notes, if I'm using these functions, what should I used instead |
timfish
commented
Sep 15, 2022
Sorry this didn't end up highlighted as a breaking change in the release notes. What are you using these exports for? |
maclockard
commented
Sep 16, 2022
I was using them to pass an event to |
AbhiPrasad
commented
Sep 16, 2022
Hey @maclockard - thanks for reaching out. Here we expect folks you to just directly use Why are you preferring to use |
Why make folks who are manually setting some of the event fields manually set all of them? What I was doing was using the event builder methods to make an event then overriding some fields manually before capturing. |
AbhiPrasad
commented
Sep 19, 2022
We recommend folks use constmyError=newError(...);Sentry.withScope((scope)=>{scope.addEventProcessor(event=>{if(badCondition){// don't send event to Sentryreturnnull;}event.X= ...
event.Y= ...
returnevent;});Sentry.captureException(myError);});We intentionally constrain the top level API surface so that we can more easily make improvements and changes without user impact. Both |
These exports aren't even used in the React Native SDK:
https://github.com/getsentry/sentry-react-native/blob/a8d5ac86e3c53c90ef8e190cc082bdac440bd2a7/src/js/backend.ts#L45-L61