Skip to content

Update RecordsWrite description#306

Closed
LiranCohen wants to merge 2 commits intomainfrom
lirancohen/records-write-updates
Closed

Update RecordsWrite description#306
LiranCohen wants to merge 2 commits intomainfrom
lirancohen/records-write-updates

Conversation

@LiranCohen
Copy link
Copy Markdown
Member

@LiranCohen LiranCohen commented Jun 12, 2024

  • Remove mention of commit strategy from RecordsWrite.
  • Update definition of parentId
  • Update description of contextId
  • Remove reference for RecordsCommit

@LiranCohen
Copy link
Copy Markdown
Member Author

I forgot that we have this PR: #257

My PR is also missing some things that the other PR addresses, (such as removing protocolVersion). I just happened to see these inconsistencies when I was doing an update for RecordsDelete and figured I'd address them.

I'm in favor of closing this PR and having the original #257 rebased and updated according to some of the comments and any subsequent changes as it's fairly old.

@andorsk andorsk closed this Aug 21, 2024
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