litaio / lita-slack

A Slack adapter for Lita.
MIT License
141 stars 130 forks source link

Support slash commands #60

Open brodock opened 8 years ago

brodock commented 8 years ago

Can we support slack slash commands?

Slash commands in Slack has some benefits over the way we are doing today:

SlackBotsy looks like support this type of integration:

https://github.com/rlister/slackbotsy/blob/master/lib/slackbotsy/bot.rb#L134

cdrage commented 8 years ago

+1

don't like how lita-slack-handler removed all the code and has forced our team to use this version with lita which doesn't support slash commands.

edit: found I could use: config.robot.alias instead for slash commands! :dancer:

jimmycuadra commented 8 years ago

My understanding is that slash commands are "swallowed" by Slack and not sent as a message to the room. Is that not the case anymore, or are you suggesting that users set up custom slash commands which talk to Lita over HTTP?

brodock commented 8 years ago

I'm suggesting that lita should enable (probably as a configuration flag) that we opt for Slack "over HTTP" builtin slash command support.

asser-hassan commented 8 years ago

Hi, revisiting that space. I wonder if that's still on the table?

brodock commented 7 years ago

@jimmycuadra slack commands work through webhooks. We can reuse puma and the existing http handler support to enable that. I believe this will require a separate definition for services that implements slash commands (like slack, mattermost).