yjs / y-protocols

Yjs encoding protocols
MIT License
109 stars 39 forks source link

Update sync.js #22

Closed aryzing closed 1 year ago

aryzing commented 2 years ago

Updated the comment. As it was written, it felt as if this library performed the actual sending of the message contained in encoder. However, this library is network agnostic.

Rather, it seems, it's up to the server using this lib whether it wants to send the message or not, and should not send it if it detects the message is empty (as seen here).

dmonad commented 1 year ago

Thank you, that makes sense!