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

alternateNick support, ERR_NICKNAMEINUSE handling, nick suffixing #14

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ardcore
Copy link

@ardcore ardcore commented Mar 7, 2014

in a case of ERR_NICKNAMEINUSE error, alternateNick is used. if it's also taken, it gets suffixed with _ until the NICK command succeeds.

showcase:

01:52 -!- mycoolbot [~mycoolbot@---] has joined #bottest
01:52 -!- mycoolerbot [~mycoolbot@---] has joined #bottest
01:52 -!- mycoolerbot_ [~mycoolbot@---] has joined #bottest

@mmalecki
Copy link
Owner

@ardcore thanks for the contribution, and sorry for the super late reply here, are you still interested in getting this in?

I like the feature, but I think we should give user more control over what nick gets chosen. Could we add an event for nick being in use, and have user call Ircb#nick on their own, and if there's no handler for that event, default to suffixing with _?

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.

2 participants