-
Notifications
You must be signed in to change notification settings - Fork 9
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
Automatically check for desyncs #16
Comments
Always open to issue from anyone! Does "Fetch Code" not do this? It's meant for resync too. Did you mean some kind of automatic resyncing? I just fixed an issue that would stop all syncs when doing a large paste. Try updating and let me know if you still get desynced often. Desyncs should be rare if everything is working as intended. |
Yes, fetch code is the manual version of what I suggest to be automated. |
Ideally things would rarely (if ever) desync, so hopefully we won't need to check for desyncs automatically. I'll think about it though, it's a good idea. |
Maybe think of it as a safety precaution. If the code desynces more rarely, it's harder to notice. |
YJS will handle all syncing in the future, so this won't be needed. Desyncs will be mathematically impossible, not just rare. :) |
It's me again!
(Are you even open for more issues from me? ;)
Sometimes when I code with my friend, we get desynced. I cant really tell when this happens, we assume it starts when multiple users do huge changes at the same time, like pasting multiple lines of code or appling extensions like a beautifier.
What I thought of now: maybe it would be a good idea to have an auto re-sync feature. One user can be assigned as the master via the room interface and when an auto re-sync happens, all other users will change their code to the master code. Just like key frames in video encryption, this would make it more stable in my totally unprofessionall opinion.
The text was updated successfully, but these errors were encountered: