aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Fixed old tests and added 2 new onesGravatar wookie1842020-09-01-0/+22
|
* Added checks to ignore webhook and bot messagesGravatar wookie1842020-09-01-0/+4
|
* Allow moderators to use defconGravatar Dennis Pham2020-08-30-6/+6
|
* Merge pull request #1118 from python-discord/bug/mod/1094/watch-leaked-tokenGravatar Dennis Pham2020-08-27-3/+9
|\ | | | | Censor watchchannel messages when they contain leaked tokens
| * Merge branch 'master' into bug/mod/1094/watch-leaked-tokenGravatar Dennis Pham2020-08-27-171/+278
| |\ | |/ |/|
* | Update Discord Partner badgeGravatar Joe Banks2020-08-27-1/+1
| |
* | Merge pull request #1124 from python-discord/feat/backend/707/alias-decoratorGravatar Shirayuki Nekomata2020-08-26-82/+88
|\ \ | | | | | | Support specifying top-level aliases in the command decorator
| * \ Merge branch 'master' into feat/backend/707/alias-decoratorGravatar Shirayuki Nekomata2020-08-26-70/+156
| |\ \ | |/ / |/| |
* | | Merge pull request #1128 from python-discord/bold_linkGravatar Shirayuki Nekomata2020-08-26-2/+2
|\ \ \ | | | | | | | | Bold link to asking guide in help channel embeds
| * | | Bold link to asking guide in embedsGravatar Boris Muratov2020-08-26-2/+2
|/ / /
* | | Merge pull request #1105 from python-discord/joseph/enhanced-user-commandGravatar kwzrd2020-08-23-68/+154
|\ \ \ | | | | | | | | Enhanced user command
| * \ \ Merge branch 'master' into joseph/enhanced-user-commandGravatar kwzrd2020-08-23-56/+96
| |\ \ \ | |/ / / |/| | |
| * | | Don't just exclude custom emoji, include the name of the emoteGravatar Joe Banks2020-08-23-1/+3
| | | |
| * | | Address review comments from @kwzrdGravatar Joe Banks2020-08-23-19/+14
| | | |
| * | | Re-align status iconsGravatar Joe Banks2020-08-16-3/+3
| | | |
| * | | Add status information to user commandGravatar Joseph Banks2020-08-14-8/+13
| | | |
| * | | Update tests for user commandsGravatar Joseph Banks2020-08-14-32/+55
| | | |
| * | | Add badges & status to user commandGravatar Joseph Banks2020-08-14-28/+67
| | | |
| * | | Add YAML values for badgesGravatar Joseph Banks2020-08-14-0/+11
| | | |
| * | | Add constants for badgesGravatar Joseph Banks2020-08-14-0/+11
| | | |
| | * | Type check root aliasesGravatar MarkKoz2020-08-24-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | Just like normal aliases, they should only be tuples or lists. This is likely done by discord.py to prevent accidentally passing a string when only a single alias is desired.
| | * | Help: include root aliases in outputGravatar MarkKoz2020-08-22-1/+3
| | | |
| | * | Include root aliases in the command name conflict testGravatar MarkKoz2020-08-22-0/+1
| | | |
| | * | Replace alias command definitions with root_aliasesGravatar MarkKoz2020-08-22-81/+15
| | | | | | | | | | | | | | | | The fruits of my labour.
| | * | Bot: fix AttributeError for commands which lack root_aliasesGravatar MarkKoz2020-08-22-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Even if the `command` decorators are patched, there are still some other internal things that need to be patched. For example, the default help command subclasses the original `Command` type. It's more maintainable to exclude root alias support for these objects than to try to patch everything.
| | * | Patch d.py decorators to support root aliasesGravatar MarkKoz2020-08-22-0/+10
| | | | | | | | | | | | | | | | | | | | | | | | To avoid explicitly specifying `cls` everywhere, patch the decorators to set the default value of `cls` to the `Command` subclass which supports root aliases.
| | * | Bot: add root alias supportGravatar MarkKoz2020-08-22-0/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Override `Bot.add_command` and `Bot.remove_command` to add/remove root aliases for a command (and recursively for any subcommands). This has to happen in `Bot` because there's no reliable way to get the `Bot` instance otherwise. Therefore, overriding the methods in `GroupMixin` unfortunately doesn't work. Otherwise, it'd be possible to avoid recursion by processing each subcommand as it got added.
| | * | Define a Command subclass with root alias supportGravatar MarkKoz2020-08-22-0/+15
| |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A subclass is used because cogs make copies of Command objects. They do this to allow multiple instances of a cog to be used. If the Command class doesn't inherently support the `root_aliases` kwarg, it won't end up being copied when a command gets copied. `Command.__original_kwargs__` could be updated to include the new kwarg. However, updating it and adding the attribute to the command wouldn't be as elegant as passing a `Command` subclass as a `cls` attribute to the `commands.command` decorator. This is because the former requires copying the entire code of the decorator to add the two lines into the nested function (it's a decorator with args, hence the nested function).
* | | Disable raw commandsGravatar Joe Banks2020-08-20-2/+2
| | |
* | | Merge pull request #1116 from Numerlor/reminders-fixGravatar Joe Banks2020-08-20-2/+24
|\ \ \ | | | | | | | | Restrict reminder methods to authors and admins.
| * \ \ Merge branch 'master' into reminders-fixGravatar Joe Banks2020-08-20-15/+8
| |\ \ \ | |/ / / |/| | |
* | | | Merge pull request #1117 from ↵Gravatar Mark2020-08-20-15/+8
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | python-discord/bug/1112/backend/wait-for-deletion-client Make client parameter mandatory for wait_for_deletion
| * | | | Make client parameter mandatory for wait_for_deletionGravatar MarkKoz2020-08-20-15/+8
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | A client instance is necessary for the core feature of this function. There is no way to obtain it from the other arguments. The previous code was wrong to think `discord.Guild.me` is an equivalent. Fixes #1112
| * / / Restrict reminder methods to authors and admins.Gravatar Numerlor2020-08-20-2/+24
|/ / / | | | | | | | | | | | | | | | Before, any user could modify the reminders of others by the id. This restricts the behaviour to only admins and users can only modify the reminders they authored.
| | * Don't send code block help if message has a webhook tokenGravatar MarkKoz2020-08-20-0/+2
| | |
| | * Watchchannels: censor message content if it has a leaked tokenGravatar MarkKoz2020-08-20-3/+7
| |/ |/| | | | | Fixes #1094
* | Merge pull request #1106 from AtieP/masterGravatar Mark2020-08-19-1/+3
|\ \ | | | | | | Add doc cleanup
| * | msg rather than doc_embedGravatar AtieP2020-08-16-2/+2
| | |
| * | Use wait_for_deletion from /bot/utils/messages.py instead of doc_cleanupGravatar AtieP2020-08-16-25/+3
| | |
| * | Satisfy some of the Azure pipelines' code requirementsGravatar AtieP2020-08-15-3/+3
| | |
| * | Add doc cleanupGravatar AtieP2020-08-15-2/+26
| | |
* | | Merge pull request #1109 from python-discord/bug/mod/1065/unnominate-bannedGravatar Mark2020-08-19-20/+36
|\ \ \ | | | | | | | | Unnominate banned users from the talent pool
| * | | Replace stinky single-item unpacking syntaxGravatar MarkKoz2020-08-19-1/+1
| | | |
| * | | Swap argument order in ChainMapsGravatar MarkKoz2020-08-19-2/+2
| | | | | | | | | | | | | | | | | | | | The defaults should be last to ensure they don't take precedence over explicitly set values.
| * | | Unnominate banned users from the talent poolGravatar MarkKoz2020-08-15-19/+35
| | |/ | |/| | | | | | | Fixes #1065
* | | Merge pull request #1100 from python-discord/bug/filters/1099/dupe-offensive-msgGravatar Senjan212020-08-19-3/+11
|\ \ \ | | | | | | | | Filtering: ignore errors for duplicate offensive messages
| * | | Filtering: ignore errors for duplicate offensive messagesGravatar MarkKoz2020-08-12-3/+11
| | |/ | |/| | | | | | | | | | | | | | | | The error happens when a filter is triggered by a message edit. Fixes #1099 Fixes BOT-6B
* | | Merge pull request #1107 from python-discord/bug/filters/1005/antispam-ctx-patchGravatar Senjan212020-08-19-4/+9
|\ \ \ | | | | | | | | Don't patch ctx.message.author in antispam
| * | | Truncate mod log contentGravatar MarkKoz2020-08-15-0/+4
| | | | | | | | | | | | | | | | Discord has a limit of 2000 characters for messages.
| * | | Don't patch ctx.message.author in antispamGravatar MarkKoz2020-08-15-4/+5
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The modification propagated across all code that is using the same `Message` object, including all other `on_message` listeners. This caused weird bugs e.g. the filtering cog thinking the bot authored a message that triggered a filter. Patching only `ctx.author` means the implementation is more fragile. Infraction code must ensure it only retrieves the author via `ctx.author` and not through `ctx.message`. Fixes #1005 Fixes BOT-7D