aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Silence: reschedule silences on startupGravatar MarkKoz2020-08-16-10/+32
| | | | | | | | | | | | Remove the moderator notification when unloading the cog because. Its purpose was to remind to manually unsilence channels. However, this purpose is now obsolete due to automatic rescheduling. The notification was buggy anyway due to a race condition with the bot shutting down, and that'd be further complicated by having to asynchronously retrieve channels from the redis cache too. Fixes #1053
* Silence: persist silenced channelsGravatar MarkKoz2020-08-16-0/+11
| | | | Can be used to support rescheduling.
* Silence: add separate unsilence error for manually-silenced channelsGravatar MarkKoz2020-08-15-1/+9
| | | | | | | It was confusing to reject a silence and an unsilence when overwrites were manually set to False. That's because it's contradictory to show an error stating it's already silence but then reject an unsilence with an error stating the channel isn't silenced.
* Silence: notify admins if previous overwrites were not cachedGravatar MarkKoz2020-08-15-12/+25
| | | | | Admins will have to manually check the default values used and adjust them if they aren't the desired values for that particular channel.
* 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.