-
-
Notifications
You must be signed in to change notification settings - Fork 58
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add spotlight: true
comment to relevant Sentry.init
snippets
#535
Comments
Hey @Lms24 should we do this? For the platforms where we spotlight is supported. |
Hey, @Shubhdeep12 yes, let's give this a try! If you take a look at this, let's please first tackle the JS frameworks with one PR per framework (to keep things a bit easier to review and revert if necessary). Let me know if you have questions around the wizard. |
Sure, i'll do it in the same way. |
Hi @Lms24 Added the PRs Please check. |
EDIT: reworded the reply because I mixed it up with #547 I'm going to close this issue because the comment-based approach was initially implemented in RN, SvelteKit and NextJS. However due to work in #622, it was removed again from NextJS. In the future, we'll make spotlight a selectable feature like tracing and replay. However, this we'll only do when the time is right. |
Our wizard should inject a commented
spotlight: true
flag to enable sending events from the SDK to the spotlight sidecar:For now, the following platforms come to mind:
Client-side is taken care of by the spotlight overlay (which injects the integration into the Sentry SDK).
Also
The text was updated successfully, but these errors were encountered: