hotfix(MessageCommand): do not allow anybody to call this command#1407
hotfix(MessageCommand): do not allow anybody to call this command#1407christolis wants to merge 1 commit intoTogether-Java:developfrom
Conversation
Signed-off-by: Chris Sdogkos <work@chris-sdogkos.com>
Zabuzard
left a comment
There was a problem hiding this comment.
voting to close this. we removed all in-code perm checks because this is something discord provides on its own and lets owners configure in much more detail and ease inside discord
if u add this u would also have to add it for /ban and all other sensitive commands.
|
There is a |
Thats slightly different I would say. It is to ensure transitive permission dependencies. So for example to ensure a user who is mod cant ban an admin, only vice versa. We removed all in-code perm checks like "/ban can only be used by users with ban permission" the moment discord added this stuff in their own preferred UI. Either case, the urgency is gone, we dont need a hotfix anymore, so we have time to think this through carefully and dont need to act immediately. |
No description provided.