Skip to content
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

fix: revert even more rrweb #21816

Merged
merged 1 commit into from
Apr 24, 2024
Merged

fix: revert even more rrweb #21816

merged 1 commit into from
Apr 24, 2024

Conversation

daibhin
Copy link
Contributor

@daibhin daibhin commented Apr 24, 2024

Problem

#21812 didn't go far enough, customers are still experiencing issues.

Changes

Remove the entire change in rrweb-io/rrweb#1401, including a now suspicious while loop (don't know why I didn't consider that the culprit sooner)

@daibhin daibhin changed the title revert even more rrweb fix: revert even more rrweb Apr 24, 2024
@daibhin daibhin mentioned this pull request Apr 24, 2024
@daibhin daibhin requested a review from a team April 24, 2024 16:10
@daibhin daibhin merged commit b4cea12 into master Apr 24, 2024
98 of 99 checks passed
@daibhin daibhin deleted the dn-fix/revert-more-rrweb branch April 24, 2024 16:12
@daibhin
Copy link
Contributor Author

daibhin commented Apr 24, 2024

Merging to get this out sooner

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant