-
Notifications
You must be signed in to change notification settings - Fork 187
Add Application Permissions Support #156
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
Merged
Merged
Changes from all commits
Commits
Show all changes
63 commits
Select commit
Hold shift + click to select a range
92cd689
Implement slash command permissions.
benwoo1110 0d6a255
Support get permission id type based on class type.
benwoo1110 4e982cb
Implement permission support for non-cog commands.
benwoo1110 00a11ba
Add support for default permissions.
benwoo1110 446cac6
Apply permissions based on applicable guilds specified.
benwoo1110 2054983
Rethink permission applicable guilds logic.
benwoo1110 9a23efd
Add option to delete perms from old guilds.
benwoo1110 e9f5570
Using logging instead of print.
benwoo1110 09003c0
Check if perms not registered before attempting to register.
benwoo1110 4249972
Add some debug print for testing.
benwoo1110 8799e97
Properly check if permissions has already been registered.
benwoo1110 5301fde
Forgot to pass permission dict as kwargs.
benwoo1110 439db8d
Fix logic issue with permission data matching checks.
benwoo1110 36f73e5
Fix wrong perm variable used in for loop.
benwoo1110 513c4d9
Use logger instead of print for debug messages.
benwoo1110 e8cdb4d
Fix some formatting issues.
benwoo1110 2de3385
Add docs to method and some cleanup.
benwoo1110 b46f348
Rename to applicable_guilds.
benwoo1110 70cc7f3
Apply some suggested code changes.
benwoo1110 9ee4b4d
Empty list instead of None for api_permissions.
benwoo1110 d927a26
Make permissions use a dict structure as suggested.
benwoo1110 ed7f8e8
Update method names to align with previous change.
benwoo1110 bee61a9
Implement generate_permissions function for easy permission creation.
benwoo1110 899534c
Flatten command_dict to minimize breaking changes.
benwoo1110 930bc01
Add coroutine for permissions requests.
benwoo1110 9a9902d
Fix issues with command syncing.
benwoo1110 315d307
Add sub command support.
benwoo1110 55bccd0
Add permission decorator.
benwoo1110 86ea0d3
Update discord_slash/utils/manage_commands.py
benwoo1110 ee61eff
Update discord_slash/utils/manage_commands.py
benwoo1110 6a62a14
Add link to discord api docs.
benwoo1110 acd2eb8
Merge branch 'permissions' of https://github.com/benwoo1110/discord-p…
benwoo1110 93b8865
Sub commands should not have permissions things.
benwoo1110 7815254
Add PermissionData class to store each ApplicationCommandPermissions.
benwoo1110 da60405
Merge permissions for subcommands.
benwoo1110 7e16da9
Rework CommandObject class inheritance.
benwoo1110 5bb9245
Add missing documentation on some functions.
benwoo1110 dca61e9
Add wiki guide for permissions.
benwoo1110 9cf6fa7
Updated docs as per suggestion.
benwoo1110 22e9e86
Add support for Cog subcomand.
benwoo1110 8001dec
Fix BasecommandObject attributes.
benwoo1110 cc63233
Merge branch 'permissions' of https://github.com/benwoo1110/discord-p…
benwoo1110 602fd6b
Fix a small typo.
benwoo1110 09a5d0d
Fix more typos.
benwoo1110 333b803
Update docs/gettingstarted.rst
benwoo1110 c725978
Update documentation.
benwoo1110 0b1449e
Rename to BaseCommandObject
benwoo1110 7c9550c
Apply suggestions from code review
benwoo1110 3526d6f
Add note on @everyone permission.
benwoo1110 8c39cec
Remove duplicate ids when on create_multi_ids_permission.
benwoo1110 89d38c0
Add guild_id parameter to GuildPermissionsData.
benwoo1110 4fd02ff
Update discord_slash/utils/manage_commands.py
benwoo1110 d1b2709
Update discord_slash/client.py
benwoo1110 24138fa
Update docs/gettingstarted.rst
benwoo1110 77e6a71
Fix minor issues when creating permissions with SlashCommandPermissio…
benwoo1110 fec21a6
Dont need to cast, should already be int.
benwoo1110 d0ab6c2
Add abit more logging on permission map.
benwoo1110 3a15ee4
fix logging issues.
benwoo1110 a650d65
Improving logging for sync_commands.
benwoo1110 323db71
Add docs on @slash.permission decorator.
benwoo1110 95f11b0
Do not need to copy guild_ids list there.
benwoo1110 63bef90
Fix issue with guild_ids is cog subcommand.
benwoo1110 54e79e7
Add support for merge permissions by cog subcommand.
benwoo1110 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.