Conversation
…n 7.x to 8.0.0 (#11731) As the User Feedback feature moves from alpha/beta to GA some changes to the public API have been made, and version requirements for the server (for on-prem users) are being solidified. Related to getsentry/develop#1206 --------- Co-authored-by: Catherine Lee <55311782+c298lee@users.noreply.github.com> Co-authored-by: Billy Vong <billyvg@users.noreply.github.com>
[Gitflow] Merge master into develop
When getting the user from the scope, the user can also be an empty object, which doesn't work with null coalescing. This checks to see if the user exists and that it's not empty in all scopes Fixes getsentry/sentry#70347
I dropped the ball on this a bit to deprecate but removing it now should still be fine.
…11998) In order to keep bundle size compact, performance integrations (except for http & fetch) have to be manually added for serverless packages. This means that users will have to do e.g. this if they want to have mysql instrumented: ```js import * as Sentry from '@sentry/aws-serverless'; Sentry.init({ integrations: [Sentry.mysqlIntegration()] }); ``` Closes #11991 --------- Co-authored-by: Lukas Stracke <lukas.stracke@sentry.io>
`snapshot` is a public API.
I think the other methods we do not need to expose (?).
With this, usage will be:
```ts
const canvas = getClient()?.getIntegrationByName<ReturnType<typeof replayCanvasIntegration>>('ReplayCanvas');
await canvas.snapshot();
```
Closes #11971
Member
Author
|
(supersedes #11948 b/c of branch name) |
Replays will fail to start recording when using `start()` specifically when manually recording and after the user has been idle for a long period of time. We need to reset the user activity state when we call `start()`, otherwise the session will be [incorrectly] considered to be idle and unable to send any replay events. Closes #11983
add section for integrations API cleanup Apply suggestions from code review Co-authored-by: Francesco Novy <francesco.novy@sentry.io> rework sdk init section update changelog
1d96065 to
2e08b05
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.


Let's 🚢 this!