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

Check eTag in Edit Mode #166

Closed
wants to merge 8 commits into from
Closed

Check eTag in Edit Mode #166

wants to merge 8 commits into from

Conversation

githubkoma
Copy link
Contributor

@githubkoma githubkoma commented Oct 17, 2021

A Proposal to address the fact that 2+ People concurrently working on the same document, are prone to overwriting each others work. (see #163 #14 #74 #120)
Now this is an attempt to check the eTag Header of the Diagram, and let the User decide on how to proceed in case of a conflict.

On Conflict between local file state <> remote file state:

  • When Autosave is enabled: Error Message warns the user and asks him to "Save" manually.
  • "Save" Button: Draft Modal opens, that..
    • ..shows the remote Versions Diagram
    • ..asks the User how to proceed:
      • Import the Remote File into the local Users Editor
      • Replace the Remote File

Needs extensive testing, help appreciated!

GIF

@githubkoma githubkoma closed this by deleting the head repository Jan 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.

1 participant