aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* 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
|
* 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.
| * | Don't ping everyone when tripping filter in DMs.Gravatar Leon Sandøy2020-07-29-1/+1
|/ / | | | | | | | | | | | | | | | | We don't need a ping in #mod-alerts whenever someone is tripping a filter (like invites or bad language) in a DM to the bot. We can still send an embed, so that we can action it, but there is no urgent need to respond if it's just a direct message to the bot. This is particularly true now that we have #dm-log.
* | Merge pull request #1023 from ks129/help-channels-pinGravatar Mark2020-07-28-0/+24
|\ \ | | | | | | Help System: Implement question message pinning
| * \ Merge branch 'master' into help-channels-pinGravatar Mark2020-07-28-481/+2396
| |\ \ | |/ / |/| |
* | | Merge pull request #1069 from python-discord/fix/pollGravatar scragly2020-07-29-2/+2
|\ \ \
| * | | correct import orderGravatar Senjan212020-07-28-1/+1
| | | |
| * | | fix poll command by using clean_content converterGravatar Senjan212020-07-28-2/+2
|/ / /
* | | Allow specifying a channel to send !embed embedsGravatar kosayoda2020-07-28-3/+7
| | |
* | | Update IDs of Code Jam rolesGravatar Sebastiaan Zeeff2020-07-27-2/+2
| | | | | | | | | | | | I've updated the IDs of the two Code Jam Roles to the newly create roles we have.
* | | Fix: Implicit string concatenation considered harmfulGravatar kosayoda2020-07-24-4/+1
| | | | | | | | | | | | | | | | | | | | | | | | Python joins two string adjacent string literals implicitly, which may cause unintended side effects when used with certain string methods. >>> 'A' ' '.join(['1', '2', '3']) '1A 2A 3'
* | | Merge pull request #1064 from python-discord/bug/1036/empty-embed-fieldsGravatar Dennis Pham2020-07-23-2/+5
|\ \ \ | | | | | | | | Check that embed desc is not Empty before stripping.
| * \ \ Merge branch 'master' into bug/1036/empty-embed-fieldsGravatar Dennis Pham2020-07-23-65/+88
| |\ \ \ | |/ / / |/| | |
* | | | Disabled burst_shared filter temporarilyGravatar Joe Banks2020-07-23-4/+0
| | | |
* | | | Merge pull request #1062 from python-discord/bug/util/897/truncate-charinfoGravatar Mark2020-07-22-21/+15
|\ \ \ \ | | | | | | | | | | Truncate charinfo results
| * \ \ \ Merge branch 'master' into bug/util/897/truncate-charinfoGravatar Mark2020-07-22-25/+234
| |\ \ \ \ | |/ / / / |/| | | |
* | | | | Merge pull request #1063 from python-discord/bug/util/jams-multi-categoriesGravatar Mark2020-07-22-40/+73
|\ \ \ \ \ | | | | | | | | | | | | Support multiple categories for code jam team creation
| * | | | | Jam tests: use the MAX_CHANNELS constant moreGravatar MarkKoz2020-07-22-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | It's clearer to write MAX_CHANNELS - 2 than a literal 48.
| * | | | | Jam tests: assert equality of new categoryGravatar MarkKoz2020-07-22-1/+2
| | | | | |
| * | | | | Jam tests: add subtests to non-existent category testGravatar MarkKoz2020-07-22-7/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The test has to account for not only the name not matching, but also a lack of available spaces for new channels.
| * | | | | Jam tests: fix category testGravatar MarkKoz2020-07-22-14/+24
| | | | | |
| * | | | | Jams: create a new category if others are fullGravatar MarkKoz2020-07-22-20/+31
|/ / / / /
| * | | | Charinfo: correct char limit used in error messageGravatar Mark2020-07-22-1/+1
| | | | | | | | | | | | | | | Co-authored-by: Kieran Siek <[email protected]>
| * | | | Charinfo: up char limit and reduce line limitGravatar MarkKoz2020-07-22-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Pagination means more characters can be supported without cluttering anything. It also means infinite lines, so there's no longer a need to squeeze out the most from a single page. Reducing the line limit leads to a smaller, tidier presentation.
| * | | | Charinfo: use more descriptive field name Gravatar Mark2020-07-22-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since the raw field is displayed on every page, but pages are incomplete, it may be unclear whether the field's value is for the current page or for all pages. Co-authored-by: Kieran Siek <[email protected]>
| * | | | Charinfo: paginate the resultsGravatar MarkKoz2020-07-21-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Pagination ensures the results will never go over the char limit for an embed. Fixes #897 Fixes BOT-3D
| * | | | Charinfo: use send_denial helperGravatar MarkKoz2020-07-21-13/+7
| | | | |
| | * | | Check that embed desc is not Empty before stripping.Gravatar kosayoda2020-07-23-2/+5
| |/ / / |/| | |
* | | | Merge pull request #957 from ks129/jam-testGravatar Mark2020-07-22-17/+193
|\ \ \ \ | |/ / / |/| | | Code Jams unit tests
| * | | Merge branch 'master' into jam-testGravatar Mark2020-07-22-869/+3594
| |\ \ \ | |/ / / |/| | |