Skip to content

fix(replay): Export Replay from Sentry namespace in full CDN bundle #7119

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.

Sign up for GitHub

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 3 commits into from
Feb 9, 2023
Merged

fix(replay): Export Replay from Sentry namespace in full CDN bundle #7119

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
2e636e7
fix(replay): Export Replay from Sentry namespace in full CDN bundle
Lms24 Feb 9, 2023
85f9b34
update changelog w/ correct sort order and PR number
Lms24 Feb 9, 2023
d7dca93
update test
Lms24 Feb 9, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@
- feat(replay): Send client_report when replay sending fails (#7093)
- fix(node): `LocalVariables`, Improve frame matching for ESM (#7049)
- fix(node): Add lru cache to http integration span map (#7064)
- fix(replay): Export Replay from Sentry namespace in full CDN bundle (#7119)

Work in this release contributed by @JamesHenry. Thank you for your contribution!

Expand Down
7 changes: 6 additions & 1 deletion packages/tracing/src/index.bundle.replay.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,11 @@ import { Replay } from '@sentry/browser';

import * as Sentry from './index.bundle';

// TODO (v8): Remove this as it was only needed for backwards compatibility
// We want replay to be available under Sentry.Replay, to be consistent
// with the NPM package version.
Sentry.Integrations.Replay = Replay;

export default Sentry;
export { Replay };

export * from './index.bundle';
3 changes: 2 additions & 1 deletion packages/tracing/test/index.bundle.replay.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import Sentry from '../src/index.bundle.replay';
import * as Sentry from '../src/index.bundle.replay';

// Because of the way how we re-export stuff for the replay bundle, we only have a single default export
const { Integrations } = Sentry;
Expand All @@ -15,5 +15,6 @@ describe('Integrations export', () => {
});

expect(Integrations.Replay).toBeDefined();
expect(Sentry.Replay).toBeDefined();
});
});