sentry-wizard icon indicating copy to clipboard operation
sentry-wizard copied to clipboard

Add `spotlight: true` comment to relevant `Sentry.init` snippets

Open Lms24 opened this issue 1 year ago • 4 comments

Our wizard should inject a commented spotlight: true flag to enable sending events from the SDK to the spotlight sidecar:

Sentry.init({
  dsn: '...',
  //...
  // uncomment this to enable Spotlight (https://spotlightjs.com):
  // spotlight: true
})

For now, the following platforms come to mind:

  • NextJS server-side
  • SvelteKit server-side
  • Remix server-side

Client-side is taken care of by the spotlight overlay (which injects the integration into the Sentry SDK).

Also

  • React Native (?)
  • Electron (?)
  • Not sure about the state in iOS/Android

Lms24 avatar Feb 13 '24 08:02 Lms24

Hey @Lms24 should we do this? For the platforms where we spotlight is supported.

Shubhdeep12 avatar Mar 05 '24 15:03 Shubhdeep12

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.

Lms24 avatar Mar 05 '24 16:03 Lms24

Sure, i'll do it in the same way.

Shubhdeep12 avatar Mar 05 '24 16:03 Shubhdeep12

Hi @Lms24 Added the PRs

Please check. Thanks

Shubhdeep12 avatar Mar 06 '24 17:03 Shubhdeep12