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 #21821

Merged
merged 1 commit into from
Apr 24, 2024
Merged

fix: revert even more rrweb #21821

merged 1 commit into from
Apr 24, 2024

Conversation

daibhin
Copy link
Contributor

@daibhin daibhin commented Apr 24, 2024

Problem

#21816 didn't work either 😢 I misapplied the patch

Changes

Added the correct code back

How did you test this code?

Very difficult to test because I cannot download recordings right now because the exported file has no snapshots (given they cannot be parsed). Hoping this will finally fix things

@daibhin daibhin requested a review from a team April 24, 2024 16:45
Copy link
Collaborator

@neilkakkar neilkakkar left a comment

Choose a reason for hiding this comment

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

zero clue what's going on here + how to validate it, but if you need an approval to merge, you got it :P

@daibhin daibhin force-pushed the dn-fix/rrweb-correct-patch branch from 59b137d to 7373801 Compare April 24, 2024 17:12
@daibhin daibhin changed the title revert even more rrweb fix: revert even more rrweb Apr 24, 2024
@daibhin daibhin enabled auto-merge (squash) April 24, 2024 17:17
@daibhin daibhin merged commit 72bf3ab into master Apr 24, 2024
98 checks passed
@daibhin daibhin deleted the dn-fix/rrweb-correct-patch branch April 24, 2024 17:29
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.

2 participants