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

support channel permission unions for plugins #711

Open
rob-3 opened this issue May 29, 2022 · 4 comments
Open

support channel permission unions for plugins #711

rob-3 opened this issue May 29, 2022 · 4 comments
Labels
enhancement New feature or request

Comments

@rob-3
Copy link
Contributor

rob-3 commented May 29, 2022

[5:45 PM] rob-3: !8ball
[5:45 PM] Lion: Please use this command in a Public channel.
Here are 3 of the 47 supported channel(s): 
general,
memes,
programming_help.

I think 8ball should work in #bot-commands. Once I finish rewriting it, I will fix this bug myself if it still exists.

@tanndlin
Copy link
Collaborator

This is intended
The channel permission is public

Channel permission is checked here at runtime

Channel types are defined by name in Constants.ts

However, I would be down for changing the way this is defined as it could be useful to have a channel with multiple types
(ie #bot_commands could be public and bot)

@joeydotdev
Copy link
Collaborator

for some historic context- we defined bot commands as not a public chan because we didn't want to start conversation in that channel/wanted to keep the messages sent in that chan strictly for class registration/garage status. we believed commands that lean more social should stick to the other public chans

@tanndlin
Copy link
Collaborator

for some historic context- we defined bot commands as not a public chan because we didn't want to start conversation in that channel/wanted to keep the messages sent in that chan strictly for class registration/garage status. we believed commands that lean more social should stick to the other public chans

Agree with this

@tanndlin
Copy link
Collaborator

tanndlin commented May 29, 2022

However, I would be down for changing the way this is defined as it could be useful to have a channel with multiple types (ie #bot_commands could be public and bot)

Or maybe less intrusively, the ability to union types for a certain plugin
Just something like an array

public permission: ChannelType = [ChannelType.Public, ChannelType.Bot];

@c650 c650 changed the title Can't run 8ball in #bot-commands support channel permission unions for plugins Jun 4, 2022
@c650 c650 added the enhancement New feature or request label Jun 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants