-
-
Notifications
You must be signed in to change notification settings - Fork 467
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
Implement application command permissions v2 #1183
Conversation
…into permissions-v2
Implement application command permissions v2
Just to be clear, what will this PR bring? What are the new changes to the permissions system? |
This allows bot devs & server managers to choose permission requirements (such as "manage messages") for commands |
Co-authored-by: Jay Turner <[email protected]>
Codecov Report
@@ Coverage Diff @@
## master #1183 +/- ##
==========================================
+ Coverage 33.42% 33.52% +0.09%
==========================================
Files 93 93
Lines 17615 17558 -57
==========================================
- Hits 5888 5886 -2
+ Misses 11727 11672 -55
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Pretty much done except for gateway events |
@krittick wanna resolve mr conflicts? |
Working on it now. |
# Conflicts: # discord/commands/core.py
Implements the new permissions system.
Original PR: #1129