Skip to content

CRDT.ORMultiMap and LWWMap serialization #602

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

Jump to bottom
Merged
merged 1 commit into from
May 1, 2020
Merged

CRDT.ORMultiMap and LWWMap serialization #602

merged 1 commit into from
May 1, 2020

Conversation

@yim-lee
Copy link
Member

@yim-lee yim-lee commented Apr 30, 2020

Motivation:
Make all CRDTs serializable. See #507.

Modifications:

  • Add ProtobufRepresentable conformance to CRDT.ORMultiMap.
  • Add ProtobufRepresentable conformance to CRDT.LWWMap.

Result:
Resolves #507

@yim-lee
CRDT.ORMultiMap and LWWMap serialization
155b6ec 
Motivation:
Make all CRDTs serializable. See #507.

Modifications:
- Add `ProtobufRepresentable` conformance to `CRDT.ORMultiMap`.
- Add `ProtobufRepresentable` conformance to `CRDT.LWWMap`.

Result:
Resolves #507
ktoso
ktoso approved these changes May 1, 2020
View reviewed changes
Copy link
Member

@ktoso ktoso left a comment

Choose a reason for hiding this comment

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

Nice thanks! <3

@ktoso ktoso merged commit 31c81d2 into apple:master May 1, 2020
@ktoso ktoso deleted the more-crdt-seri branch May 1, 2020 01:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ktoso ktoso ktoso approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Codable conformances for CRDT types

2 participants

@yim-lee @ktoso