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

Roles Module #66

Open
anna328p opened this issue Oct 7, 2019 · 5 comments
Open

Roles Module #66

anna328p opened this issue Oct 7, 2019 · 5 comments
Labels
feat Feature suggestion/plan module Sub-label to use along with others

Comments

@anna328p
Copy link

anna328p commented Oct 7, 2019

Brainstorming area for now.

Admin

role list

role self enable Anime # Possibly in config

role give Muted @snug#0001
role give Muted @snug#0001 1w1d
role remove Muted @snug#0001

User

role self list

role self get Anime
role self get Anime 1w1d
role self remove Anime

Aliases (possibly in config)

alias mute="role give Muted"
alias unmute="role remove Muted"

alias litemute="role give lite-muted"
alias unlitemute="role remove lite-muted"

mute @snug#0001
unmute @snug#0001

litemute @bitmapper#7863
unlitemute @bitmapper#7863 

Notes

  • Config may be in the config but that would might annoying
  • IDs instead of names a possiblility
  • Obsoletes mute module
@ldelelis
Copy link
Member

ldelelis commented Oct 7, 2019

I'd keep the "roles" module for aesthetic/low functionality roles, considering mutes has a certain amount of logic behind it, as to avoid having such a bloated module.

@ldelelis ldelelis added feat Feature suggestion/plan module Sub-label to use along with others labels Oct 7, 2019
@anna328p
Copy link
Author

anna328p commented Oct 7, 2019

some other notes:

  • current mute module's logic could be adapted fairly easily for other roles
  • self-roles would be basically an alias for role give @me with a flag to enable for specific roles

@kayew
Copy link

kayew commented Oct 7, 2019

For self-roles, including a self-ping seems unnecessary. Most bots just have you do role <role name> and it'll be assigned to the person that ran the command.

@anna328p
Copy link
Author

anna328p commented Oct 7, 2019

For self-roles, including a self-ping seems unnecessary. Most bots just have you do role <role name> and it'll be assigned to the person that ran the command.

In the original post, I had some examples of how self-roles would work:

role self list

role self get Anime
role self get Anime 1w1d
role self remove Anime

@kayew
Copy link

kayew commented Oct 8, 2019

I had some examples of how self-roles would work

Yeah, this seems like the best way to assign roles. I didn't see this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat Feature suggestion/plan module Sub-label to use along with others
Projects
None yet
Development

No branches or pull requests

3 participants