-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
docs(javascript): Add CDN bundle example for performance #2039
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/sentry/sentry-docs/mg460r202/sentry.dev |
# Using yarn | ||
$ yarn add @sentry/browser @sentry/tracing | ||
|
||
# Using npm | ||
$ npm install @sentry/browser @sentry/tracing | ||
``` | ||
|
||
Alternatively, instead of `npm` packages, you can use our pre-built CDN bundle that combines both `@sentry/browser` and `@sentry/tracing`: | ||
|
||
<JsCdnTag apm={true} /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wasn't sure how to get the tabbed experience while using this <JsCdnTag/>
component. Open to ideas if someone can direct me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for anyone else: you cant, what ben did here is the right solution
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Haven't tried using tags+tabs yet, maybe Cramer will be best bet if there's a better way to do it.
I noticed the Vercel deployment is kinda broken, every page loads and after a quick blink renders nothing but Loading (StaticQuery)
. Guessing it is unrelated to the changes here.
im gonna update this and merge |
Instructions are now tabbed between ESM and CDN.