smogon / Porygon-Z

The Bot for the Official Smogon Discord (WIP)
https://discord.gg/smogon
MIT License
13 stars 6 forks source link

Work out a permissions system #7

Closed KrisXV closed 5 years ago

KrisXV commented 5 years ago

There's currently not a permissions system and when one is eventually created, should it be done on a by-role basis or user-specific basis or what? (Conversation below for context)

[2:17 PM] Kris: is anyone working on the permission system for the bot actively or n? i have some ways to do it in discord.js [2:17 PM] chaos: which bot? [2:18 PM] Kris: porygon-z [2:19 PM] chaos: unless @HoeenHero has uncommited changes, i do not think so [2:19 PM] Kris: i saw acomment about looking into handling permissions earlier so i thought i should offer to help [2:19 PM] Kris: (comment in the code) [2:19 PM] HoeenHero: Added to the project board under design (we need to decide how to do it before doing it) [2:19 PM] HoeenHero: If you want to create an issue to discuss it go for it

HoeenCoder commented 5 years ago

You mentioned you have some ideas? I currently don't have anything planned for this, but the permissions should probably be based on discord permissions bar a "bot admin" position based on discord user snowflakes (user id number) for things like !eval.