aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| * 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
| * Merge branch 'master' into whitelist_systemGravatar Leon Sandøy2020-08-03-135/+1821
| |\ | |/ |/|
* | Remove superfluous Available help channels.Gravatar Leon Sandøy2020-07-31-4/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds a little bit of logic to the Help Channel `init_available` coroutine, which runs when the cog loads. This ensures that if there are more help channels in available than there should be, we remove the superfluos ones. Previously, if the bot started with too many channels, it would maintain and defend that excessive amount. This is because we never actually count the number of channels before adding in new available channels whenever one disappears. If we ever get too many available channels in the future, this can be solved by simply reloading this cog.
* | Merge pull request #1073 from python-discord/fix/webhook_removerGravatar Leon Sandøy2020-07-30-1/+1
|\ \ | | | | | | Change regex so it catches new discord URL
| * | Use full flag name for case-insensitivityGravatar Senjan212020-07-30-1/+1
| | | | | | | | | | | | requested by lemon
| * | Missed `?` in regex.Gravatar Senjan212020-07-30-1/+1
| | |
| * | Use non-capturing group instead.Gravatar Senjan212020-07-30-1/+1
| | |
| * | Change regex so it catches new discord URLGravatar Senjan212020-07-30-1/+1
|/ /
* | Merge pull request #1072 from python-discord/fix/message_pinGravatar Sebastiaan Zeeff2020-07-30-0/+2
|\ \ | | | | | | Fix channel moving incase `message.pin` fails
| * | Fix channel moving incase `message.pin` failsGravatar Senjan212020-07-30-0/+2
|/ /
* | Merge pull request #1071 from python-discord/no_mod_alert_ping_for_dmsGravatar Dennis Pham2020-07-29-1/+1
|\ \ | | | | | | Don't ping everyone in #mod-alerts when tripping filter via DMs.