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 optional username for IRC backend. #256

Merged
merged 1 commit into from Jun 22, 2014
Merged

Conversation

aadis
Copy link
Contributor

@aadis aadis commented Jun 22, 2014

For servers where nickname and username are different, allow passing in
a different username. If username is not supplied, it's defaulted to
nickname.

For servers where nickname and username are different, allow passing in
a different username. If username is not supplied, it's defaulted to
nickname.
gbin added a commit that referenced this pull request Jun 22, 2014
Allow optional username for IRC backend.
@gbin gbin merged commit edeec08 into errbotio:master Jun 22, 2014
@gbin
Copy link
Member

gbin commented Jun 22, 2014

Thx for splitting it up !

@aadis
Copy link
Contributor Author

aadis commented Jun 22, 2014

How about the other one (Unicode error)?

That was a while ago. I dug inside the XML parser and there seemed no easy way to fix it. So reluctantly I made the exception hack.

@gbin
Copy link
Member

gbin commented Jun 22, 2014

I think the fix should be in the construction of various Message() in irc.py. Here we should make sure it is already converted correctly to unicode there.

zoni pushed a commit to zoni/errbot that referenced this pull request Jun 24, 2014
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.

None yet

2 participants