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

irc bot doesn't reply #20

Closed
honza opened this issue Oct 25, 2011 · 11 comments
Closed

irc bot doesn't reply #20

honza opened this issue Oct 25, 2011 · 11 comments

Comments

@honza
Copy link

honza commented Oct 25, 2011

I can see the bot in the room. When I do:

hubot: image me pony

Hubot receives it ok, but it fails when it tries to send the message back to the room.

Ideas?

@honza
Copy link
Author

honza commented Oct 25, 2011

It replies with:

ERROR: err_notexttosend: hubot No text to send

@atmos
Copy link
Contributor

atmos commented Oct 25, 2011

Hey Honza,

I haven't used the IRC adapter, I think @xpaulbettsx has and said he has some changes coming down the line.

@honza
Copy link
Author

honza commented Oct 25, 2011

Awesome. I'm happy to help debug.

@anaisbetts
Copy link
Contributor

I'll hack on this tonight, see if I can fix the glitches as well as add IDENTIFY support

Paul Betts

SENT FROM MY COMMODORE 64: RESPONSES MAY BE IN ALL CAPS

On Oct 25, 2011, at 13:33, Corey Donohoereply@reply.github.com wrote:

Hey Honza,

I haven't used the IRC adapter, I think @xpaulbettsx has and said he has some changes coming down the line.

Reply to this email directly or view it on GitHub:
#20 (comment)

@kennethreitz
Copy link

I got it to run today, but it wasn't pretty — looks like 'user' is undefined in the 'send' method, so I hardcoded the IRC channel, and all was well.

If I knew more CoffeeScript I'd send a pull request with a real fix.

@simensen
Copy link

I can confirm that 5b291a1 works.

@honza
Copy link
Author

honza commented Oct 25, 2011

Same here. Thanks @smoak

peterwooley added a commit to peterwooley/hubot that referenced this issue Oct 26, 2011
…updates the test user (though, I've been unable to get the tests running on my box).
@peterwooley
Copy link

So, this was the fix I was working on, which basically just keeps the user's name in the Robot.User's options. Plus, it might fix a test, if I were able to get the tests running locally.

@atmos
Copy link
Contributor

atmos commented Oct 26, 2011

Fixed in 076420a

@atmos atmos closed this as completed Oct 26, 2011
@honza
Copy link
Author

honza commented Oct 26, 2011

@xpaulbettsx Any progress on the IDENTIFY stuff? Thanks

@anaisbetts
Copy link
Contributor

There's a PR for it, waiting on a small change before merging it

Paul Betts

SENT FROM MY COMMODORE 64: RESPONSES MAY BE IN ALL CAPS

On Oct 26, 2011, at 12:16, Honza Pokornyreply@reply.github.com wrote:

@xpaulbettsx Any progress on the IDENTIFY stuff? Thanks

Reply to this email directly or view it on GitHub:
#20 (comment)

glucero pushed a commit to ProCern/hubot that referenced this issue Aug 20, 2012
technicalpickles pushed a commit that referenced this issue Apr 17, 2023
feat(hubot): #20 Hubot is blocking people from handling SIGTERM in their code
technicalpickles pushed a commit that referenced this issue Apr 17, 2023
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

No branches or pull requests

6 participants