Skip to content

feat(replay): Update replay CDN usage docs #6088

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

Merged
merged 5 commits into from
Jan 19, 2023
Merged

feat(replay): Update replay CDN usage docs #6088

merged 5 commits into from
Jan 19, 2023

Conversation

mydea
Copy link
Member

@mydea mydea commented Jan 17, 2023

This updates the CDN usage docs to use the new full replay bundle by default, streamlining this.

While at it, I also reordered the "CDN installation" page generally a bit:

  1. I moved the tracing bundle to top, as that is the default experience we want to provide.
  2. After that, I added a new tracing + replay section
  3. After that, there is the error only bundle
  4. Moved the "Updates to naming scheme in SDK version 7" section further down, as it is not really very up to date anymore

Related to #6082, and dependent on getsentry/sentry-javascript#6818.

Closes getsentry/sentry-javascript#6573
Closes #5972

@mydea mydea requested a review from a team as a code owner January 17, 2023 12:10
@mydea mydea self-assigned this Jan 17, 2023
@vercel
Copy link

vercel bot commented Jan 17, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
sentry-docs ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Jan 19, 2023 at 4:09PM (UTC)

Copy link
Member

@Lms24 Lms24 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good and I like the simplification from the previous version of the page! I just had a couple of suggestions for wording but feel free to go with whatever you think makes sense.


Version 7 of the Sentry JavaScript SDKs changed the bundles to be ES6 by default.
Previously, the default bundles were compiled to ES5. If you need to support ES5, see [Available Bundles](#available-bundles) or the [Migration Guide to Version 7](https://github.com/getsentry/sentry-javascript/blob/master/MIGRATION.md#upgrading-from-6x-to-7x).
To use Sentry for error monitoring, performance tracing, and Replay, you can use the following bundle:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
To use Sentry for error monitoring, performance tracing, and Replay, you can use the following bundle:
To use Sentry for error and performance monitoring, as well as Session Replay, you can use the following bundle:

@Lms24
Copy link
Member

Lms24 commented Jan 17, 2023

Added #5972 to the list of issues closed by this PR

@mydea mydea force-pushed the fn/replay-cdn-docs branch from 4974e9f to 47c9ff7 Compare January 19, 2023 15:51
@mydea mydea merged commit c48600e into master Jan 19, 2023
@mydea mydea deleted the fn/replay-cdn-docs branch January 19, 2023 16:17
@github-actions github-actions bot locked and limited conversation to collaborators Feb 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create a single CDN bundle with browser+tracing+replay [JS] Make Browser+Tracing CDN bundle the main bundle
3 participants