aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* PEP: Removed `while` loop from refresh checking on `get_pep_embed`Gravatar ks1292020-05-18-11/+9
|
* PEP Improvisations: Remove PEP URLs refreshing task + replace it with new systemGravatar ks1292020-05-17-12/+18
| | | | Now PEP command request PEP listing when PEP is not found and last refresh was more time ago than 30 minutes instead task.
* PEP Improvisations: Fix cache if statementGravatar ks1292020-05-17-1/+1
| | | | Add `not` in check is key exist in cache.
* PEP Improvisations: Move error embed to variables instead creating on `ctx.send`Gravatar ks1292020-05-17-6/+5
|
* PEP Improvisations: Remove response from logging to avoid newlineGravatar ks1292020-05-17-2/+1
|
* PEP Improvisations: Fix formatting of blocks Gravatar ks1292020-05-17-0/+2
| | | | | Added newline before logging after indention block. Co-authored-by: Mark <[email protected]>
* PEP Improvisations: Simplify cache item check on `async_cache` decoratorGravatar ks1292020-05-17-2/+1
| | | Co-authored-by: Mark <[email protected]>
* PEP Improvisations: Move errors sending from PEP command to `get_pep_embed`Gravatar ks1292020-05-15-9/+14
| | | | Before this, all error embeds was returned on `get_pep_embed` but now this send this itself and return only correct embed to make checking easier in command.
* PEP Improvisations: Fix `get_pep_embed` docstringGravatar ks1292020-05-15-1/+1
|
* PEP Improvisations: Fix `get_pep_zero_embed` docstringGravatar ks1292020-05-15-1/+1
|
* PEP Improvisations: Move `get_pep_zero_embed` back to CogGravatar ks1292020-05-15-15/+15
| | | | Moved `get_pep_zero_embed` back to the cog, but made this `staticmethod`.
* PEP Improvisations: Fix log text formattingGravatar ks1292020-05-15-2/+4
| | | | Use repo own alignment of multiline text.
* PEP Improvisations: Remove unnecessary typehintGravatar ks1292020-05-15-1/+0
| | | | Removed unnecessary type hint that I used for IDE and what I forget to remove.
* PEP Improvisations: Fix importsGravatar ks1292020-05-14-1/+1
| | | Replace `in_channel` with `in_whitelist`. This mistake was made to merge conflicts.
* Merge branch 'master' into pep-improvisationsGravatar ks1292020-05-14-269/+800
|\
| * Merge pull request #899 from ks129/python-newsGravatar Sebastiaan Zeeff2020-05-13-55/+321
| |\ | | | | | | Python News implemention
| | * Merge branch 'master' into python-newsGravatar Sebastiaan Zeeff2020-05-13-219/+522
| | |\ | | |/ | |/|
| * | Expand guild whitelistGravatar S. Co12020-05-12-1/+7
| | |
| * | Merge pull request #924 from ↵Gravatar Daniel Brown2020-05-11-20/+58
| |\ \ | | | | | | | | | | | | | | | | python-discord/feature/hemlock/perma-ban-override-temp Perma Bans now Overwrite Temp Bans
| | * \ Merge branch 'master' into feature/hemlock/perma-ban-override-tempGravatar Daniel Brown2020-05-11-58/+34
| | |\ \ | | |/ / | |/| |
| * | | Merge pull request #938 from Suhail6inkling/remind_aliasGravatar Leon Sandøy2020-05-11-1/+1
| |\ \ \ | | | | | | | | | | Add remindme alias for the remind command
| | * | | Add remindme alias for the remind commandGravatar Suhail2020-05-10-1/+1
| |/ / /
| * | | Merge pull request #923 from python-discord/feat/util/remove-mention-cmdGravatar Daniel Brown2020-05-08-56/+2
| |\ \ \ | | | | | | | | | | Remove the mention command and configuration settings for it
| | * \ \ Merge branch 'master' into feat/util/remove-mention-cmdGravatar Daniel Brown2020-05-08-1/+31
| | |\ \ \ | | |/ / / | |/| | |
| * | | | Merge pull request #903 from python-discord/bug/backend/894/win-selector-loopGravatar Daniel Brown2020-05-07-1/+7
| |\ \ \ \ | | | | | | | | | | | | Use selector event loop on Windows
| | * \ \ \ Merge branch 'master' into bug/backend/894/win-selector-loopGravatar Daniel Brown2020-05-07-136/+405
| | |\ \ \ \ | | |/ / / / | |/| | | |
| * | | | | Merge pull request #919 from ↵Gravatar Sebastiaan Zeeff2020-05-05-0/+12
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | python-discord/bug/backend/911/log-listener-exceptions Log unhandled errors from event listeners
| | * \ \ \ \ Merge branch 'master' into bug/backend/911/log-listener-exceptionsGravatar Sebastiaan Zeeff2020-05-05-3/+16
| | |\ \ \ \ \ | | |/ / / / / | |/| | | | |
| * | | | | | Merge pull request #925 from Savant-Dev/antimalwareGravatar Daniel Brown2020-05-05-0/+12
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | Update to Antimalware Filter (.txt uploads)
| | * \ \ \ \ \ Merge branch 'master' into antimalwareGravatar Daniel Brown2020-05-05-2/+2
| | |\ \ \ \ \ \ | | |/ / / / / / | |/| | | | | |
| | * | | | | | Update antimalware to filter txt files in cases where messages were longer ↵Gravatar Savant-Dev2020-05-04-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | than 2000 chars
| | * | | | | | Update extension filter to distinguish .txt in cases where messages are ↵Gravatar Savant-Dev2020-05-01-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | longer than 2000 characters
| | * | | | | | Update extension filter to distinguish .txt in cases where messages are ↵Gravatar Savant-Dev2020-05-01-0/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | longer than 2000 characters
| | | * | | | | Log unhandled errors from event listenersGravatar MarkKoz2020-04-29-0/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | By default, discord.py prints them to stderr. To better help detect such errors in production, they should instead be logged with an appropriate log level. Some sentry metadata has also been included. `on_error` doesn't work as a listener in a cog so it's been put in the Bot subclass. Fixes #911
| | | | * | | | Fix awaiting non-coroutine when closing the statsd transportGravatar MarkKoz2020-04-24-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `BaseTransport.close()` is not a coroutine and therefore should not be awaited.
| | | | * | | | Use selector event loop on WindowsGravatar MarkKoz2020-04-22-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | aiodns requires the selector event loop for asyncio. In Python 3.8, the default event loop for Windows was changed to proactor. To fix this, the event loop is explicitly set to selector.
| | | | | * | | Remove mention command constantsGravatar MarkKoz2020-05-02-10/+0
| | | | | | | |
| | | | | * | | Remove the mention commandGravatar MarkKoz2020-05-01-46/+2
| | |_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It was made obsolete by a new Discord feature. Users can be granted a permission to mention a role despite the role being set as non-mentionable.
| | | | | * | Restructure `apply_ban()` logic Gravatar Daniel Brown2020-05-11-12/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Another refactor/cleaning to make the logic clearer and easier to understand. Also cleaned up the trace logs to be shorter and more concise. Thanks, @scragly! Co-authored-by: scragly <[email protected]>
| | | | | * | apply_ban() logic refinedGravatar Daniel Brown2020-05-07-15/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Refined the logic for `apply_ban()` even further to be cleaner. (Thanks, @MarkKoz!) Signed-off-by: Daniel Brown <[email protected]>
| | | | | * | Addressing Review ChangesGravatar Daniel Brown2020-05-07-25/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Changed docstring explanation and function name of `get_active_infractions` to `get_active_infraction()` to better convey that only one infraction is returned. Also changed all relevant uses to reflect that change. - Added explanation of parameter `send_msg` to the doc strings of `pardon_infraction()` and `get_active_infraction()` - Adjusted placement of `log.trace()` in `pardon_infraction()` - Adjusted logic in `apply_ban()` to remove redundant check. - Adjusted logic in `apply_ban()` to be consistent with other checks. Signed-off-by: Daniel Brown <[email protected]>
| | | | | * | Merge branch 'master' into feature/hemlock/perma-ban-override-tempGravatar Daniel Brown2020-05-04-2/+2
| | | | | |\ \ | | |_|_|_|/ / | |/| | | | |
| * | | | | | Merge pull request #922 from python-discord/bug/info/914/user-animated-avatarGravatar kwzrd2020-05-01-2/+2
| |\ \ \ \ \ \ | | |/ / / / / | |/| | | | | Display animated avatars in the user info command
| | * | | | | Tests: change avatar_url_as assertion to use static_formatGravatar MarkKoz2020-04-30-1/+1
| | | | | | |
| | * | | | | Display animated avatars in the user info commandGravatar MarkKoz2020-04-30-1/+1
| |/ / / / / | | | | | | | | | | | | | | | | | | Fixes #914
| | | | * / Perma Bans now Overwrite Temp BansGravatar Daniel Brown2020-05-04-18/+55
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Changed `has_active_infraction` to `get_active_infractions` in order to add additional logic in `apply_ban`. - Added `send_msg` parameters to `pardon_infraction` and `get_active_infractions` so that multi-step checks and actions don't need to send additional messages unless told to do so. Signed-off-by: Daniel Brown <[email protected]>
| * | | | Run a category check before logging that we are checking for an answered ↵Gravatar Joseph Banks2020-04-30-1/+2
| |/ / / | | | | | | | | | | | | help channel
| * | | Merge pull request #913 from python-discord/add-bottom-sorting-to-help-channelsGravatar Sebastiaan Zeeff2020-04-28-12/+63
| |\ \ \ | | | | | | | | | | Sort help channels and add support for `how-to-get-help` channel
| | * | | Log ID of member who claimed a help channelGravatar Sebastiaan Zeeff2020-04-28-0/+1
| | | | |
| | * | | Add option to ingore channels in help categoriesGravatar Sebastiaan Zeeff2020-04-28-3/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As we want to add an "informational" channel to the `Python Help: Available` category, we need to make sure that the Help Channel System ignores that channel. To do that, I've added an `is_excluded_channel` staticmethod that returns `True` if a channel is not a TextChannel or if it's in a special EXCLUDED_CHANNELS constant. This method is then used in the method that yields help channels from a category and in the `on_message` event listener that determines if a channel should be moved from `Available` to `Occupied`.