search

yjs / y-webrtc

WebRTC Connector for Yjs
MIT License
458 stars 111 forks source link

Update y-webrtc.js #16

Closed disarticulate closed 3 years ago

disarticulate commented 3 years ago

If there can be only one, better to just destroy the other one. rooms is an internal var and can't be accessed anywhere else, so it just strands a room if theres some other error.

dmonad commented 3 years ago

If you open two connections to the same room, you probably did something wrong. What happens to the old Yjs document? This is a breaking change that I won't merge. Although, I agree, it would be nice to have two Yjs document connected to the same room. This should be possible somehow. But there is some implementations overhead that I'm unwilling to fix because I don't have a usecase for it.