Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

[Feature Request] Add a way to remove groups #163

Closed
goldfire opened this Issue · 4 comments

3 participants

@goldfire

Since groups stay in memory even after everyone in the group disconnects (or at least this is the behavior I've been experiencing), you could end up with millions of groups sitting around in memory with nobody connected. Adding something like the below would be helpful.

nowjs.removeGroup(groupName);

@Peterfly

yes, it sounds like a good improvement. but then only the server side should be allowed to call it. otherwise, a client may easily disconnect other clients in the same group.

@goldfire

Yes, good point.

@Peterfly

already refined it on the branch removegroup. it may come out later

@ericz

Hi goldfire,

This functionality is now in the github master. It'll be released with the next version on npm.

If you're on the github master, you can use nowjs.removeGroup(groupName);

Thanks to @steveWang @Peterfly for helping with this.

@ericz ericz closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.