Question: "room" in socket.io seems can emit message to multiple clients. Have we consider that ?
Andrei Alecu
@andreialecu
it would add more complexity than the current solution
you still have to authenticate users joining rooms somehow
I looked into it in the past and wasn't feasible
at least not to replace what we have, but maybe to enhance it further down the road
emitToUsers() was pretty much unusable and broken in real world use before this PR
Eric Fong
@ericfong
npm test passed. need more time to digest the code.
Andrei Alecu
@andreialecu
@ericfong see private
Andrei Alecu
@andreialecu
deployd/deployd#573 I will merge this if there are no objections, tested in modern browsers to be ok, and other browsers should respect min-height if they can't understand that height
hey guys! I would like to start helping with maintaining deployd but I need somewhere to start. any ideas?
I mean, how do you guys work? (git, git flow, PR to master…) where can I find docs on the architecture of deployd? (folders, injection, ctx, etc…) and what would you have me do? I would love to start helping with the docs but for that I need to know where are the docs right now and how to edit them, etc...
Rares Golea
@rgolea
sorry… my bad… just found out where are the docs...