aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
* | | | | 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.
* | | Merge pull request #1084 from ↵Gravatar Mark2020-08-09-1/+4
|\ \ \ | | | | | | | | | | | | | | | | python-discord/bug/filters/1027/ignore-webhook-names Ignore webhooks for nickname filter
| * \ \ Merge branch 'master' into bug/filters/1027/ignore-webhook-namesGravatar Dennis Pham2020-08-09-136/+155
| |\ \ \ | |/ / / |/| | |
* | | | Merge pull request #1085 from python-discord/bug/util/1079/long-poll-titlesGravatar Dennis Pham2020-08-09-0/+2
|\ \ \ \ | | | | | | | | | | Utils: show error message for long poll titles
| * \ \ \ Merge branch 'master' into bug/util/1079/long-poll-titlesGravatar Dennis Pham2020-08-09-136/+153
| |\ \ \ \ | |/ / / / |/| | | |
* | | | | Remove unnecessary edits during paginationGravatar Joe Banks2020-08-08-10/+0
| | | | |
* | | | | Merge pull request #1086 from python-discord/bug/help/1082/unpin-id-noneGravatar Dennis Pham2020-08-07-27/+64
|\ \ \ \ \ | | | | | | | | | | | | Help channel message pin fixes
| * \ \ \ \ HelpChannels: fix conflict in move_to_dormantGravatar MarkKoz2020-08-07-100/+90
| |\ \ \ \ \ | |/ / / / / |/| | | | |
* | | | | | Merge pull request #1092 from python-discord/bug/help/1074/duplicate-claimGravatar scragly2020-08-07-3/+1
|\ \ \ \ \ \
| * \ \ \ \ \ Merge branch 'master' into bug/help/1074/duplicate-claimGravatar scragly2020-08-07-98/+90
| |\ \ \ \ \ \ | |/ / / / / / |/| | | | | |
| * | | | | | HelpChannels: clear claimant cache when channel goes dormantGravatar MarkKoz2020-08-05-3/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The claimed channel check in `on_message` relies on the cache being cleared when a channel goes dormant. If it's not cleared, it will think the channel is still in use.
| | * | | | | HelpChannels: simplify control flow in is_emptyGravatar MarkKoz2020-08-07-4/+2
| | | | | | |
| | * | | | | HelpChannels: add logging to is_emptyGravatar MarkKoz2020-08-04-0/+3
| | | | | | |
| | * | | | | HelpChannels: more accurate empty checkGravatar MarkKoz2020-08-04-5/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The bot's pin message was being picked up as the last message, so the system was not considering the channel empty.
| | * | | | | HelpChannels: create separate function to pin a messageGravatar MarkKoz2020-08-04-9/+7
| | | | | | |
| | * | | | | HelpChannels: create a generic function to handle pin errorsGravatar MarkKoz2020-08-04-11/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This can be used for both pinning and unpinning messages. The error handling code was largely similar between them.
| | * | | | | HelpChannels: don't unpin message if ID is NoneGravatar MarkKoz2020-08-04-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes #1082 Fixes BOT-7G
| | * | | | | HelpChannels: add more detail to unpin log messagesGravatar MarkKoz2020-08-04-3/+7
| | | | | | |
| | * | | | | HelpChannels: move unpinning to separate functionGravatar MarkKoz2020-08-04-12/+16
| | | | | | |
| | | * | | | Utils: show error message for long poll titlesGravatar MarkKoz2020-08-04-0/+2
| | |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Embeds have a maximum length of 256 for titles. Fixes #1079 Fixes BOT-7Q
| | | * / / Filtering: ignore webhooks for nickname filterGravatar MarkKoz2020-08-04-1/+4
| | |/ / / | | | | | | | | | | | | | | | Fixes #1027