[feature request] listing channels
scotticles opened this issue · 8 comments
would be nice to have a /list command.
Thanks.
https://tools.ietf.org/html/rfc2812#section-3.2.6 IRC spec for /list command in case someone wants to implement it
Hmm, interesting. I always assumed /list accepted arguments such as /list >500 to retrieve channels with more than 500 users. But according to that document this is not part of the spec at all.
This would require some changes in the slate/slate-irc library we use though, as far as i can tell it's not implemented there yet.
Also, would you prefer to see this in some new window, or should we just return this in the network channel, like the MOTD? That last option would be easier to implement, but definitely not as user friendly.
@dubzi I think it could be implemented shout-side, but I can't think of a reason not to make an upstream PR.
What do we think about protecting our users with potentially destructive commands? I was testing this myself, and ran a /list
on freenode. Without any arguments, that command works, and is valid, but will return every channel on freenode. It crashed my browser 😨
IMO we just shouldn't accept that command without any arguments. Perhaps returning an example of arguments you can use when you run it without arguments?
I also found an answer to my question earlier about the spec. Basically the custom arguments such as /list >500
are handled on the irc server's side. Not sure which servers accept it since it's not part of the spec, but I assume most do.
This issue was moved to thelounge/thelounge#130
Didn't mean to close this, sorry, https://github-issue-mover.appspot.com/ did it automatically, duh...