Skip to content

feat(replay): Upgrade rrweb packages to 2.28.0 #13732

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

Jump to bottom
Merged
merged 1 commit into from
Oct 9, 2024
Merged

feat(replay): Upgrade rrweb packages to 2.28.0 #13732

merged 1 commit into from
Oct 9, 2024

Conversation

billyvg
Copy link
Member

@billyvg billyvg commented Sep 19, 2024

Pulls in upstream rrweb commits, see getsentry/rrweb#220

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Sep 19, 2024 •
edited
Loading

size-limit report 📦

Path Size % Change Change
@sentry/browser 22.73 KB - -
@sentry/browser - with treeshaking flags 21.53 KB - -
@sentry/browser (incl. Tracing) 34.97 KB - -
@sentry/browser (incl. Tracing, Replay) 71.62 KB +0.1% +73 B 🔺
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 62.03 KB +0.13% +79 B 🔺
@sentry/browser (incl. Tracing, Replay with Canvas) 75.97 KB +0.11% +85 B 🔺
@sentry/browser (incl. Tracing, Replay, Feedback) 88.73 KB +0.1% +86 B 🔺
@sentry/browser (incl. Tracing, Replay, Feedback, metrics) 90.59 KB +0.09% +83 B 🔺
@sentry/browser (incl. metrics) 27 KB - -
@sentry/browser (incl. Feedback) 39.87 KB - -
@sentry/browser (incl. sendFeedback) 27.38 KB - -
@sentry/browser (incl. FeedbackAsync) 32.17 KB - -
@sentry/react 25.49 KB - -
@sentry/react (incl. Tracing) 37.94 KB - -
@sentry/vue 26.9 KB - -
@sentry/vue (incl. Tracing) 36.86 KB - -
@sentry/svelte 22.87 KB - -
CDN Bundle 24.05 KB - -
CDN Bundle (incl. Tracing) 36.76 KB - -
CDN Bundle (incl. Tracing, Replay) 71.38 KB +0.12% +85 B 🔺
CDN Bundle (incl. Tracing, Replay, Feedback) 76.7 KB +0.11% +83 B 🔺
CDN Bundle - uncompressed 70.53 KB - -
CDN Bundle (incl. Tracing) - uncompressed 109.04 KB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 221.4 KB +0.1% +223 B 🔺
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 234.62 KB +0.1% +223 B 🔺
@sentry/nextjs (client) 37.91 KB - -
@sentry/sveltekit (client) 35.56 KB - -
@sentry/node 124.5 KB - -
@sentry/node - without tracing 93.64 KB - -
@sentry/aws-serverless 103.3 KB +0.01% +2 B 🔺

View base workflow run

@billyvg billyvg marked this pull request as ready for review September 19, 2024 16:50
@billyvg billyvg requested a review from a team as a code owner September 19, 2024 16:50
@billyvg
Copy link
Member Author

billyvg commented Sep 19, 2024

Going to deploy a prerelease to run on Sentry while I'm out next week

@billyvg billyvg force-pushed the chore-replay-upgrade-rrweb-2.28.0 branch from 125c9d4 to 1217534 Compare September 20, 2024 13:43
@billyvg billyvg mentioned this pull request Oct 1, 2024
Merged
billyvg added a commit to getsentry/sentry that referenced this pull request Oct 7, 2024
@chargome @billyvg
chore(deps): Bump sentry sdk to 8.31.0-beta.0 (#77938)
0485948 
To test getsentry/sentry-javascript#13732

---------

Co-authored-by: Billy Vong <[email protected]>
@billyvg billyvg force-pushed the chore-replay-upgrade-rrweb-2.28.0 branch 3 times, most recently from 55306b7 to 7f4e02a Compare October 9, 2024 16:03
@billyvg billyvg force-pushed the chore-replay-upgrade-rrweb-2.28.0 branch from 7f4e02a to a813533 Compare October 9, 2024 17:22
@billyvg billyvg merged commit d7c33a2 into develop Oct 9, 2024
143 of 147 checks passed
@billyvg billyvg deleted the chore-replay-upgrade-rrweb-2.28.0 branch October 9, 2024 19:31
@q1blue q1blue mentioned this pull request Oct 31, 2024
Open
@bagulm123 bagulm123 mentioned this pull request Nov 2, 2024
Open
This was referenced Nov 2, 2024
Closed
Closed
@kenn7575 kenn7575 mentioned this pull request Nov 3, 2024
Closed
@mbreadmachine mbreadmachine mentioned this pull request Nov 5, 2024
Merged
@q1blue q1blue mentioned this pull request Nov 11, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@c298lee c298lee c298lee approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@billyvg @c298lee