aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Silence: abort silence if there's already a scheduled taskGravatar MarkKoz2020-08-15-1/+1
| | | | | | Overwrites can be edited during a silence, which can result in the overwrites check failing. Checking the scheduler too ensures that a duplicate silence won't be scheduled.
* Silence: revoke permissions to add reactionsGravatar MarkKoz2020-08-15-11/+27
| | | | | | | | | | No longer assume default values for the overwrites which are modified. Save and restore previous values `add_reactions` and `send_messages` via redis. When unsilencing, check if a channel is silenced via the redis cache rather than the channel's current overwrites to ensure the task is cancelled even if overwrites were manually edited.
* 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
| |\ \ \ \ | |/ / / / |/| | | |
* | | | | Merge pull request #1093 from python-discord/discord.py-1.4Gravatar kwzrd2020-08-06-79/+51
|\ \ \ \ \ | | | | | | | | | | | | Deps: update discord.py to 1.4.0
| * | | | | Deps: update discord.py to 1.4.0Gravatar MarkKoz2020-08-06-79/+51
|/ / / / / | | | | | | | | | | | | | | | It was released on PyPI. No longer need to clone via git.
* | | | | Guild invite regex: Add support for dashes in the invite codeGravatar Dennis Pham2020-08-05-1/+1
| | | | |
* | | | | Merge pull request #1087 from ↵Gravatar Dennis Pham2020-08-05-1/+14
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | python-discord/bug/backend/1080/cog-reload-cancel-scheduler Cancel scheduled tasks when cogs unload
| * \ \ \ \ Merge branch 'master' into bug/backend/1080/cog-reload-cancel-schedulerGravatar Dennis Pham2020-08-05-16/+23
| |\ \ \ \ \ | |/ / / / / |/| | | | |
* | | | | | Merge pull request #1091 from python-discord/kwzrd/fix-1090Gravatar Dennis Pham2020-08-05-9/+8
|\ \ \ \ \ \ | | | | | | | | | | | | | | RedisCache: remove erroneous `_redis` alias
| * \ \ \ \ \ Merge branch 'master' into kwzrd/fix-1090Gravatar Dennis Pham2020-08-05-7/+15
| |\ \ \ \ \ \ | |/ / / / / / |/| | | | | |
* | | | | | | Merge pull request #1088 from python-discord/bug/util/1083/source-dynamic-objGravatar Mark2020-08-05-7/+15
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Source: raise BadArgument for dynamically-created objects
| * | | | | | | Source: remove redundant check for help commandsGravatar MarkKoz2020-08-04-4/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The code is identical to the else block and there's no reason for help commands to have an explicit check.
| * | | | | | | Source: raise BadArgument for dynamically-created objectsGravatar MarkKoz2020-08-04-3/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commands, cogs, etc. created via internal eval won't have a source file associated with them, making source retrieval impossible. Fixes #1083 Fixes BOT-7K
* | | | | | | | Revert "HelpChannels: use more reliable check for claimed channel"Gravatar kwzrd2020-08-05-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 59c62162
* | | | | | | | Merge pull request #1089 from python-discord/bug/help/1074/duplicate-claimGravatar kwzrd2020-08-05-1/+1
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | HelpChannels: use more reliable check for claimed channel
| | * | | | | | Redis: remove erroneous `_redis` aliasGravatar kwzrd2020-08-05-9/+8
| |/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If a RedisCache instance was being accessed before bot has created the `redis_cache` instance, the `_redis` alias was being set to None, causing AttributeErrors in lookups. See: #1090
| | * | | | | Cancel scheduled tasks when cogs unloadGravatar MarkKoz2020-08-04-1/+14
| |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When cogs reload, they used new Scheduler instances, which aren't aware of previously scheduled tasks. This led to duplicate scheduled tasks when cogs re-scheduled tasks upon initialisation. Fixes #1080 Fixes BOT-7H
| | * | | | 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: use more reliable check for claimed channelGravatar MarkKoz2020-08-04-1/+1
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Using the channel's category isn't reliable since it may take Discord a while to actually move the channel once it's received a request from the bot. I suppose using redis technically has the same problem, but it should be much faster and less susceptible to lag than Discord. Fixes #1074
| * | | 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
* Revert "Disabled burst_shared filter temporarily"Gravatar Joseph Banks2020-08-04-0/+4
| | | | This reverts commit be14db91b1c70993773e67cfa663fef0cfa85666.
* Merge pull request #1078 from python-discord/support_plural_filterlist_typesGravatar Dennis Pham2020-08-03-5/+17
|\ | | | | Add support for plural FilterList types.
| * Add support for plural FilterList types.Gravatar Leon Sandøy2020-08-03-5/+17
|/ | | | | | | | | This will allow mods to use '!whitelist get guild_invites' in addition to '!whitelist get guild_invite' This is just a naive implementation which works if the plural form is a simple s at the end of the word. It's implemented into the converter.
* Add some feedback to the _sync_data helper.Gravatar Leon Sandøy2020-08-03-7/+15
| | | | | | | Previously, this would not provide any feedback at all, which is really terrible UX. Sorry about that. This also adds error handling in case the API call fails.
* Move function params to 4-space indentation.Gravatar Leon Sandøy2020-08-03-2/+17
|
* Merge pull request #1058 from python-discord/whitelist_systemGravatar Leon Sandøy2020-08-03-460/+727
|\ | | | | FilterLists: Manage whitelisting and blacklisting via the bot