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

Add ignore classname & block config #25

Closed
wants to merge 2 commits into from

Conversation

sammdec
Copy link
Contributor

@sammdec sammdec commented Dec 30, 2018

This is the second part to this PR, the first part is (rrweb-io/rrweb-snapshot#6) the rrweb-snapshot should be published first then this one will need to have the first as a dependency.

Copy link
Member

@Yuyz0112 Yuyz0112 left a comment

Choose a reason for hiding this comment

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

There is also a hard code block class name in the utils.ts

@Yuyz0112
Copy link
Member

Yuyz0112 commented Apr 1, 2019

@samjbmason sorry for the late, since there are some conflicts in this patch, I added the feature in c04e4bf .

@Yuyz0112 Yuyz0112 closed this Apr 1, 2019
lewgordon-amplitude added a commit to lewgordon-amplitude/rrweb that referenced this pull request Oct 15, 2024
…ialization-fix

AMP-101866 textarea serialization fix
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