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

Support multi-user undo and redo #640

Closed
wants to merge 4 commits into from
Closed

Conversation

hyemmie
Copy link
Contributor

@hyemmie hyemmie commented Sep 11, 2023

What this PR does / why we need it: Modification of the protocol buffer is required to test the multi-user undo and redo feature being developed in the js-sdk(yorkie-team/yorkie-js-sdk#635). This change is necessary to test the pushPullChanges function without errors when the new operation(EditReverse) is added, but this draft PR does not need to be merged immediately. The js-sdk test was done by building a separate docker image locally and this draft PR is only used to mention testable conditions within comments in the js-sdk source code.

Which issue(s) this PR fixes:

Fixes #49

Special notes for your reviewer:

Does this PR introduce a user-facing change?:


Additional documentation:


Checklist:

  • Added relevant tests or not required
  • Didn't break anything

@hyemmie hyemmie added enhancement 🌟 New feature or request protocol changed 📝 Whether the protocol has changed labels Sep 11, 2023
@hyemmie hyemmie self-assigned this Sep 11, 2023
@hyemmie hyemmie closed this Sep 15, 2023
@hyemmie
Copy link
Contributor Author

hyemmie commented Sep 15, 2023

This draft PR was replaced by the PR #644.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement 🌟 New feature or request protocol changed 📝 Whether the protocol has changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support Multi-User Undo/Redo
1 participant