Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Add support for server presets. #296

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants

jdve commented Feb 16, 2014

To make it even easier for non-technical people to get started with IRC,
this change adds support for presets defined in config.js. In such a
deployment, instead of requiring users to type in all the server
configuration parameters, subway automatically connects to the preset
server and joins a predefined set of channels. After that, users may
join and leave channels as normal, but they cannot connect to other
servers.

Collaborator

hermansc commented Feb 16, 2014

A great idea and nice implementation! I love the focus on making subway more accessible to non-technical people.

I have a question though with regards to the presets.password variable and its meaning? I mean, it is obviously not referring to a channel password - since one can preset multiple channels? And we don't want to preset user-password either, now do we? :-)

Also, when we are talking about passwords. It is not possible to preset a channel with a password - or it is possible if one writes channels: ["#channelname password", "#anotherchannel foobar"], but perhaps a solution such as:

channels: [{ name: "#channelname", password: "password"}, {name: "#anotherchannel", password: ""}] 

is better?

jdve commented Feb 16, 2014

@hermansc, thanks for the feedback! Yes, I agree, presets.password really doesn't make any sense. I've removed that now. Also, AFAIK, the channels: ["#channelname password", "#anotherchannel foobar"] syntax will work, because under-the-hood it's just calling the same "join" method that we use when manually joining a channel.

Collaborator

hermansc commented Feb 16, 2014

Hey, no problem, nice to see people contributing to this project. As I said, I know it will work writing it #channel password, I just thought it would perhaps be more verbose and clearer if one separated them and did the join in the background. But, as you say there is perhaps not much point in it :-)

Add support for server presets.
To make it even easier for non-technical people to get started with IRC,
this change adds support for presets defined in config.js.  In such a
deployment, instead of requiring users to type in all the server
configuration parameters, subway automatically connects to the preset
server and joins a predefined set of channels.  After that, users may
join and leave channels as normal, but they cannot connect to other
servers.
Owner

thedjpetersen commented May 9, 2014

This is really neat. A quick note: I have worked on refactoring the client trying to make it more robust and easier to maintain the frontend code(using bower and grunt). If you would consider porting this work over to the refactor branch I would love to get it in here. Sorry that the work for this branch was lost.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment