Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow multiple networks per user #14

Closed
wants to merge 1 commit into from

Conversation

rschuetzler
Copy link

With ZNC version 1.0 or greater, you can connect one user to multiple
networks using the password: "user/network:password". This patch
provides that functionality using the network slug as the network name.

With ZNC version 1.0 or greater, you can connect one user to multiple
networks using the password: "user/network:password". This patch
provides that functionality using the network slug as the network name.
@strugee
Copy link

strugee commented May 24, 2015

@sshirokov can we get this merged? With the release of Debian Jessie this is hitting me pretty hard

@sshirokov
Copy link
Owner

@strugee I'm still running an ancient ZNC, so I can't quite test this easily. If you can tell me that this PR works correctly, I'll go ahead and merge this.

Sorry for the (super long) delay on this. 😿

@strugee
Copy link

strugee commented May 29, 2015

@sshirokov yeah, I went ahead and patched it myself in my .emacs.d and it seems to be working great. That being said, I don't know if this is compatible with old ZNCs.

@strugee
Copy link

strugee commented May 29, 2015

@sshirokov if you end up writing some code to handle older versions, I'm happy to test patches. Sadly, my ELisp-fu is not (yet) good enough to do it myself. :/

@rschuetzler
Copy link
Author

It turns out this isn't really necessary. By putting the username as "name/slug", you can get the same effect without breaking backward compatibility with <1.0 versions of ZNC.

@strugee
Copy link

strugee commented Nov 12, 2015

@rschuetzler or we could have a checkbox in Customize that lets you select your ZNC version...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants