Skip to content

Remove INode (node.__sn) and use Mirror as source of truth #868

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 13 commits into from
Apr 6, 2022
Merged

Remove INode (node.__sn) and use Mirror as source of truth #868

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
50785be
Move ids to weakmap
Juice10 Mar 31, 2022
39bf03e
Fix typo
Juice10 Mar 31, 2022
c6199f5
Move from INode to storing serialized data in mirror
Juice10 Apr 4, 2022
4b30d1f
Update packages/rrweb-snapshot/src/rebuild.ts
Juice10 Apr 5, 2022
fa5dfeb
Remove unnessisary `as Node` typecastings
Juice10 Apr 5, 2022
5d357c5
Remove unnessisary `as unknown as ...`
Juice10 Apr 5, 2022
f1a62c5
Remove unnessisary `as unknown as ...`
Juice10 Apr 5, 2022
8ce3861
Reset mirror when recording starts
Juice10 Apr 5, 2022
4e0c85c
API has changed for snapshot, change test to reflect that
Juice10 Apr 5, 2022
07de96d
Allow for es5 compatibility
Juice10 Apr 5, 2022
b5f4642
Remove unnessisary as unknown as ... and change test to reflect the A…
YunFeng0817 Apr 6, 2022
443e841
Merge branch 'master' into node-id-map
YunFeng0817 Apr 6, 2022
c44ddd8
Refactor mirror to remove `nodeIdMap`
Juice10 Apr 6, 2022
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
Prev Previous commit
Next Next commit
Fix typo
  • Loading branch information
@Juice10
Juice10 committed Mar 31, 2022
commit 39bf03ed83d3b40396a8ad6c0541662d73c89e56
2 changes: 1 addition & 1 deletion packages/rrweb-snapshot/src/snapshot.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -411,7 +411,7 @@ function serializeNode(
} = options;
// Only record root id when document object is not the base document
let rootId: number | undefined;
const docId = nodeIdMap.get(n);
const docId = nodeIdMap.get(doc);
if (docId) {
rootId = docId === 1 ? undefined : docId;
}
Expand Down