aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| * | | | | 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.
* | | 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
* | | | Merge pull request #1108 from ↵Gravatar Mark2020-08-18-0/+3
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | python-discord/bug/frontend/1030/reddit-title-escape Escape Markdown in reddit post titles
| * \ \ \ Merge branch 'master' into bug/frontend/1030/reddit-title-escapeGravatar Mark2020-08-18-9/+0
| |\ \ \ \ | |/ / / / |/| | | |
* | | | | Remove the !ask tagGravatar Leon Sandøy2020-08-16-9/+0
| |/ / / |/| | |
| * | | Escape Markdown in reddit post titlesGravatar MarkKoz2020-08-15-0/+3
|/ / / | | | | | | | | | | | | | | | | | | Use a Unicode look-alike character to replace square brackets, since they'd otherwise interfere with the Markdown. Fixes #1030
* | | Merge pull request #1102 from AtieP/patch-1Gravatar kwzrd2020-08-13-1/+1
|\ \ \ | |/ / |/| | Fix typo on the traceback tag
| * | Fix typo on the traceback tagGravatar AtieP2020-08-13-1/+1
|/ / | | | | See issue #1101
* | Merge pull request #1097 from Numerlor/remove-api-endpointsGravatar Dennis Pham2020-08-11-33/+4
|\ \ | | | | | | Remove api endpoint config values.
| * \ Merge branch 'master' into remove-api-endpointsGravatar Dennis Pham2020-08-11-0/+3
| |\ \ | |/ / |/| |
* | | Create kindling-projects tagGravatar Dennis Pham2020-08-09-0/+3
| | |
| * | Reorder site URL constants.Gravatar Numerlor2020-08-10-2/+4
| | |
| * | Remove unused api endpoint config constants.Gravatar Numerlor2020-08-10-31/+0
|/ / | | | | | | | | The constants aren't used anywhere in the bot, and are incompatible with the APIClient.