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

TypeError: Cannot set property 'channel' of undefined #77

Closed
nathanboktae opened this issue Jan 5, 2016 · 1 comment
Closed

TypeError: Cannot set property 'channel' of undefined #77

nathanboktae opened this issue Jan 5, 2016 · 1 comment

Comments

@nathanboktae
Copy link

Trying to use this project with just the basic hello responder, I ran into this error twice.

/Users/nblack/Dropbox/acronym-bot/node_modules/botkit/lib/Slackbot_worker.js:359
      msg.channel = src.channel;
                  ^

TypeError: Cannot set property 'channel' of undefined
    at Object.bot.reply (/Users/nblack/Dropbox/acronym-bot/node_modules/botkit/lib/Slackbot_worker.js:359:19)
    at Object.<anonymous> (/Users/nblack/Dropbox/acronym-bot/bot.js:25:9)
    at Object.<anonymous> (/Users/nblack/Dropbox/acronym-bot/node_modules/botkit/lib/CoreBot.js:655:20)
    at Object.Botkit.botkit.trigger (/Users/nblack/Dropbox/acronym-bot/node_modules/botkit/lib/CoreBot.js:684:41)
    at Object.<anonymous> (/Users/nblack/Dropbox/acronym-bot/node_modules/botkit/lib/SlackBot.js:499:26)
    at Object.Botkit.botkit.trigger (/Users/nblack/Dropbox/acronym-bot/node_modules/botkit/lib/CoreBot.js:684:41)
    at /Users/nblack/Dropbox/acronym-bot/node_modules/botkit/lib/CoreBot.js:755:16
    at Object.bot.findConversation (/Users/nblack/Dropbox/acronym-bot/node_modules/botkit/lib/Slackbot_worker.js:391:5)
    at Object.Botkit.botkit.receiveMessage (/Users/nblack/Dropbox/acronym-bot/node_modules/botkit/lib/CoreBot.js:751:9)
    at WebSocket.<anonymous> (/Users/nblack/Dropbox/acronym-bot/node_modules/botkit/lib/Slackbot_worker.js:101:19)

The severe lack of tests for this botkit worries me. $1.5M should get you a lot more than 5 tests.

@nathanboktae
Copy link
Author

nm I was forgettting the mesage as the first parameter. Still, CI integration and more tests is desired.

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

1 participant