idkclub / dicebot

🎲 /roll support for Slack
https://dicebot.idk.club
The Unlicense
53 stars 20 forks source link

Channel integration #21

Closed tommyhaggerty closed 7 years ago

tommyhaggerty commented 7 years ago

Hey, Nice app, it is a lot of fun. Would you be able to add integration to allow it to only deployed to a specific channel like many other apps?

Thanks!

arkie commented 7 years ago

It should only respond to /slash commands wherever they're entered. Do you mean limiting to a certain channel that it would respond in?

On Mon, 15 May 2017 at 14:50 Tommy Haggerty notifications@github.com wrote:

Hey, Nice app, it is a lot of fun. Would you be able to add integration to allow it to only deployed to a specific channel like many other apps?

Thanks!

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/arkie/hackyslack2/issues/21, or mute the thread https://github.com/notifications/unsubscribe-auth/AAOOGmGotz9Mu76gaGb73j911blUFUzyks5r6MihgaJpZM4Nbvcr .

tommyhaggerty commented 7 years ago

Yeah, sorry I wasn't more clear. That is correct. Other apps have some functionality to only allow the commands in a selected channel.  Thanks for the response.  Tommy Haggerty

Sent from my T-Mobile 4G LTE Device -------- Original message --------From: Jon Caldwell notifications@github.com Date: 5/15/17 22:02 (GMT-07:00) To: arkie/hackyslack2 hackyslack2@noreply.github.com Cc: Tommy Haggerty tommy.haggerty@gmail.com, Author author@noreply.github.com Subject: Re: [arkie/hackyslack2] Channel integration (#21) It should only respond to /slash commands wherever they're entered. Do you

mean limiting to a certain channel that it would respond in?

On Mon, 15 May 2017 at 14:50 Tommy Haggerty notifications@github.com

wrote:

Hey,

Nice app, it is a lot of fun. Would you be able to add integration to

allow it to only deployed to a specific channel like many other apps?

Thanks!

—

You are receiving this because you are subscribed to this thread.

Reply to this email directly, view it on GitHub

https://github.com/arkie/hackyslack2/issues/21, or mute the thread

https://github.com/notifications/unsubscribe-auth/AAOOGmGotz9Mu76gaGb73j911blUFUzyks5r6MihgaJpZM4Nbvcr

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/arkie/hackyslack2","title":"arkie/hackyslack2","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/arkie/hackyslack2"}},"updates":{"snippets":[{"icon":"PERSON","message":"@arkie in #21: It should only respond to /slash commands wherever they're entered. Do you\nmean limiting to a certain channel that it would respond in?\n\nOn Mon, 15 May 2017 at 14:50 Tommy Haggerty \u003cnotifications@github.com\u003e\nwrote:\n\n\u003e Hey,\n\u003e Nice app, it is a lot of fun. Would you be able to add integration to\n\u003e allow it to only deployed to a specific channel like many other apps?\n\u003e\n\u003e Thanks!\n\u003e\n\u003e —\n\u003e You are receiving this because you are subscribed to this thread.\n\u003e Reply to this email directly, view it on GitHub\n\u003e \u003chttps://github.com/arkie/hackyslack2/issues/21\u003e, or mute the thread\n\u003e \u003chttps://github.com/notifications/unsubscribe-auth/AAOOGmGotz9Mu76gaGb73j911blUFUzyks5r6MihgaJpZM4Nbvcr\u003e\n\u003e .\n\u003e\n"}],"action":{"name":"View Issue","url":"https://github.com/arkie/hackyslack2/issues/21#issuecomment-301669329"}}}

arkie commented 7 years ago

Based on clicking around https://api.slack.com/docs/slack-button it seems like the channel permissions are more for incoming webhook based apps instead of command ones? I'm not sure there's a good way to have the normal install for the command while also optionally allowing limiting to one channel, so closing out for now, but feel free to reopen with more examples or relevant docs / explanation.