archived-bot icon indicating copy to clipboard operation
archived-bot copied to clipboard

Misleading info from permission management commands feedback

Open RealKC opened this issue 7 years ago • 5 comments

img

When a user/role is already added to a permission level(for example: DJ), trying to add it to DJ will say that the user/role is not found, while it should say that the user/role is already added to that permission level, something like "User/role is already DJ!"

RealKC avatar Jul 24 '17 21:07 RealKC

This also annoyed me and Ruby. We wanted to add Ruby to a list, and I renamed myself so we would at least have a duplicate search result. But it looks like it skips already processed users, so it added a person that happened to fulfill the search query.

Shredder121 avatar Jul 24 '17 21:07 Shredder121

Also something related to this is that ;;admin list will not list the owner and roles/members that have the Administrator permission, despite them having the ADMIN permissions(since by default owners and members/roles with Administrator permission have all bot permissions).

Possible new title for issue: "Wrong/missing info from permission management commands"?

RealKC avatar Jul 25 '17 09:07 RealKC

The simple solution would be to say "No new roles and users" in place of "Nothing"

freyacodes avatar Aug 01 '17 09:08 freyacodes

To clarify, this is not a bug. This has to do with user experience.

freyacodes avatar Aug 01 '17 09:08 freyacodes

This issue is partly solved in #331.

This is what's left:

Also something related to this is that ;;admin list will not list the owner and roles/members that have the Administrator permission, despite them having the ADMIN permissions(since by default owners and members/roles with Administrator permission have all bot permissions).

I wonder how this should be accomplished? I fear we would have to check each member of a guild for those permissions. What if there are many admins/(djs/users) (think 1000, just an example), how are we going to show them?

schnapster avatar Oct 06 '17 08:10 schnapster