Elite-Kode / bgsbot

A Discord bot to track Elite Dangerous BGS
Apache License 2.0
17 stars 6 forks source link

Suggestion: Allow multiple BGS Manager Roles for the non-admin commands #52

Open Admiralfeb opened 3 years ago

Admiralfeb commented 3 years ago

Hi, We use the bot in our server.

I'm wondering if it would be possible to adjust the BGSRole to be able to assign multiple roles to it so we don't have to have one over-arching role for it? I see that BGSAdmin can be assigned to multiple roles, why not the BGS Manager/Role?

SayakMukhopadhyay commented 3 years ago

Well, to make it short, its very difficult to make this change now since the DB is designed to handle a single role for the BGSRole. It was designed to restrict permissions to execute BGS commands like adding a monitored system/faction and the idea was that a certain person or a group of people would be allowed to do that. What I could do is create a new role called the UserRole or something similar and give it access to generate reports at the minimum.