From 9aecf51f3d579a69b2df3baadd81d2c0c657962d Mon Sep 17 00:00:00 2001
From: Katie Byers <lobsterkatie@gmail.com>
Date: Tue, 15 Mar 2022 23:38:39 -0700
Subject: [PATCH 1/3] add info about available bundles

---
 src/platforms/javascript/common/install/cdn.mdx | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/src/platforms/javascript/common/install/cdn.mdx b/src/platforms/javascript/common/install/cdn.mdx
index d91be981399336..bb54f3fd39558b 100644
--- a/src/platforms/javascript/common/install/cdn.mdx
+++ b/src/platforms/javascript/common/install/cdn.mdx
@@ -51,7 +51,15 @@ Sentry.init({
 
 ## Available Bundles
 
-A variety of bundles are provided which are optimized for Sentry's various integrations.
+Our CDN hosts a variety of bundles:
+
+- `@sentry/browser` (named `bundle.<modifiers>.js`)
+- `@sentry/browser` and `@sentry/tracing` together (named `bundle.tracing.<modifiers>.js`)
+- each of the integrations in `@sentry/integrations` (named `<integration-name>.<modifiers>.js`)
+
+Each bundle is offered in both ES5 and ES6 versions, and for each version there are three bundle varieties: unminified, minified, and minified with debug logging. (That last version can be helpful for times when you need to debug an issue which only occurs in prod. In dev, it makes most sense to use the unminified bundle, which always includes logging.)
+
+So, for example, `bundle.tracing.es6.debug.min.js` is `@sentry/browser` and `@sentry/tracing` bundled together, compiled to ES6, and minified (but with debug logging included). On the simpler end of the naming spectrum, `bundle.js` is `@sentry/browser`, compiled to ES5 but not minified (with debug logging included, as it is for all unminified bundles).
 
 <JsBundleList />
 

From e5166a45043e3f658ba27a2f0ddc52f4166c7b4e Mon Sep 17 00:00:00 2001
From: Katie Byers <lobsterkatie@gmail.com>
Date: Tue, 15 Mar 2022 23:38:58 -0700
Subject: [PATCH 2/3] do random wordsmithing

---
 src/platforms/javascript/common/install/cdn.mdx | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/platforms/javascript/common/install/cdn.mdx b/src/platforms/javascript/common/install/cdn.mdx
index bb54f3fd39558b..b4187fdd396a37 100644
--- a/src/platforms/javascript/common/install/cdn.mdx
+++ b/src/platforms/javascript/common/install/cdn.mdx
@@ -6,7 +6,7 @@ description: "Learn how Sentry supports loading its JavaScript SDK via a CDN."
 
 import JsBundleList from "~src/components/jsBundleList";
 
-Sentry supports loading its JavaScript SDK via a CDN. Generally we suggest using our npm package (`@sentry/browser`) as utilizing a CDN create scenarios where Sentry is unable to load due to networking issues or common extensions like ad blockers. If you _must_ use a CDN, take a look at [loading Sentry lazily with our JS loader](../lazy-load-sentry/).
+Sentry supports loading the JavaScript SDK via a CDN. Generally we suggest using our npm package (`@sentry/browser`) instead, as using the CDN can create scenarios where Sentry is unable to load due to networking issues or common extensions like ad blockers. If you _must_ use a CDN, take a look at [loading Sentry lazily with our JS loader](../lazy-load-sentry/), which provides a deferred version of our minified ES5 browser bundle. To see what other bundles are available, see [Available Bundles](#available-bundles) below.
 
 ```html {tabTitle: CDN}
 <script
@@ -18,7 +18,7 @@ Sentry supports loading its JavaScript SDK via a CDN. Generally we suggest using
 
 ## Performance Bundle
 
-To use Sentry's performance tracing an alternative bundle is needed. This allows us to keep the filesize down for users who only need error monitoring.
+To use Sentry's performance tracing, an alternative bundle is needed. This allows us to keep the filesize down for users who only need error monitoring.
 
 ```html {tabTitle: CDN}
 <script
@@ -34,12 +34,12 @@ You only need to load `bundle.tracing.min.js`, which provides both error and per
 
 </Note>
 
-The most important thing to note here is that `Sentry.Integrations` has been made available, and can be referenced in your call to `Sentry.init`:
+Once you've included the Sentry SDK bundle in your page, you can use Sentry in your own bundle:
 
 ```javascript {tabTitle: CDN}
 Sentry.init({
   dsn: "___PUBLIC_DSN___",
-  // this assumes your build process sets "npm_package_version" in the env
+  // this assumes your build process replaces `process.env.npm_package_version` with a value
   release: "my-project-name@" + process.env.npm_package_version,
   integrations: [new Sentry.BrowserTracing()],
 

From 9fa8752f6dc52954d1d8e52b7df3c7e2be1b6f1a Mon Sep 17 00:00:00 2001
From: Katie Byers <lobsterkatie@gmail.com>
Date: Tue, 22 Mar 2022 11:59:03 -0700
Subject: [PATCH 3/3] code review fixes

---
 src/platforms/javascript/common/install/cdn.mdx | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/src/platforms/javascript/common/install/cdn.mdx b/src/platforms/javascript/common/install/cdn.mdx
index b4187fdd396a37..be1e6a6e07edde 100644
--- a/src/platforms/javascript/common/install/cdn.mdx
+++ b/src/platforms/javascript/common/install/cdn.mdx
@@ -6,7 +6,7 @@ description: "Learn how Sentry supports loading its JavaScript SDK via a CDN."
 
 import JsBundleList from "~src/components/jsBundleList";
 
-Sentry supports loading the JavaScript SDK via a CDN. Generally we suggest using our npm package (`@sentry/browser`) instead, as using the CDN can create scenarios where Sentry is unable to load due to networking issues or common extensions like ad blockers. If you _must_ use a CDN, take a look at [loading Sentry lazily with our JS loader](../lazy-load-sentry/), which provides a deferred version of our minified ES5 browser bundle. To see what other bundles are available, see [Available Bundles](#available-bundles) below.
+Sentry supports loading the JavaScript SDK from a CDN. Generally we suggest using our npm package (`@sentry/browser`) instead, as using the CDN can create scenarios where Sentry is unable to load due to networking issues or common extensions like ad blockers. If you _must_ use a CDN, take a look at [loading Sentry lazily with our JS loader](../lazy-load-sentry/), which provides a deferred version of our minified ES5 browser bundle. To see what other bundles are available, see [Available Bundles](#available-bundles) below.
 
 ```html {tabTitle: CDN}
 <script
@@ -57,9 +57,13 @@ Our CDN hosts a variety of bundles:
 - `@sentry/browser` and `@sentry/tracing` together (named `bundle.tracing.<modifiers>.js`)
 - each of the integrations in `@sentry/integrations` (named `<integration-name>.<modifiers>.js`)
 
-Each bundle is offered in both ES5 and ES6 versions, and for each version there are three bundle varieties: unminified, minified, and minified with debug logging. (That last version can be helpful for times when you need to debug an issue which only occurs in prod. In dev, it makes most sense to use the unminified bundle, which always includes logging.)
+Each bundle is offered in both ES5 and ES6 versions, and for each version there are three bundle varieties: unminified, minified, and minified with debug logging. (That last version can be helpful for times when you need to debug an issue which only occurs in production. In a development environment, it makes most sense to use the unminified bundle, which always includes logging.)
 
-So, for example, `bundle.tracing.es6.debug.min.js` is `@sentry/browser` and `@sentry/tracing` bundled together, compiled to ES6, and minified (but with debug logging included). On the simpler end of the naming spectrum, `bundle.js` is `@sentry/browser`, compiled to ES5 but not minified (with debug logging included, as it is for all unminified bundles).
+For example:
+
+- `bundle.js` is `@sentry/browser`, compiled to ES5 but not minified, with debug logging included (as it is for all unminified bundles)
+- `rewriteframes.es6.min.js` is the `RewriteFrames` integration, compiled to ES6 and minified, with no debug logging
+- `bundle.tracing.es6.debug.min.js` is `@sentry/browser` and `@sentry/tracing` bundled together, compiled to ES6 and minified, with debug logging included
 
 <JsBundleList />