aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Add Steam gift card scam to domain blacklistGravatar S. Co12020-05-18-0/+2
|
* [bug] Adjustment to changes in #941, return message sent by webhook so ↵Gravatar Joseph Banks2020-05-19-1/+1
| | | | publish can take place
* Merge pull request #941 from ks129/reddit-publishGravatar Dennis Pham2020-05-18-1/+7
|\ | | | | Add message publishing to Reddit cog
| * Merge branch 'master' into reddit-publishGravatar Dennis Pham2020-05-18-777/+1183
| |\ | |/ |/|
* | Merge pull request #858 from python-discord/decorator-factory-mutability-tagGravatar kwzrd2020-05-17-0/+37
|\ \ | | | | | | Add mutability.md tag
| * \ Merge branch 'master' into decorator-factory-mutability-tagGravatar kwzrd2020-05-17-890/+2365
| |\ \ | |/ / |/| |
* | | Merge pull request #949 from python-discord/help-command-fix-invocationGravatar kwzrd2020-05-17-41/+33
|\ \ \ | | | | | | | | Use `send_help` to ensure that our help command is correctly invoked
| * | | Use `Command`-object for `send_help`Gravatar Sebastiaan Zeeff2020-05-17-21/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As @mathsman5133 pointed out, it's better to use the `Command`-instance we typically already have in the current context than to rely on parsing the qualified name again. The invocation is now done as: `await ctx.send_help(ctx.command)`
| * | | Use `send_help` to invoke command helpGravatar Sebastiaan Zeeff2020-05-17-40/+32
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After the refactoring of the help command, we need to use the built-in method of calling the help command: `Context.send_help`. As an argument, the qualified name (a string containing the full command path, including parents) of the command can be passed. Examples: - await ctx.send_help("reminders edit") This would send a help embed with information on `!reminders edit` to the Context. - await ctx.send_help(ctx.command.qualified_name) This would extract the qualified name of the command, which is the full command path, and send a help embed to Context. - await ctx.send_help() This will send the main "root" help embed to the Context.
* | | Merge pull request #519 from mathsman5133/help-refactorGravatar Sebastiaan Zeeff2020-05-17-460/+269
|\ \ \ | | | | | | | | Refactor the !help command.
| * \ \ Merge branch 'master' into help-refactorGravatar Sebastiaan Zeeff2020-05-17-1/+1
| |\ \ \ | |/ / / |/| | |
* | | | Merge pull request #946 from vivax3794/masterGravatar Leon Sandøy2020-05-17-1/+1
|\ \ \ \ | | | | | | | | | | add "solved" as a alias for "closed"
| * | | | added "solved" as a alias for "closed"Gravatar vivax37942020-05-17-1/+1
|/ / / /
| * | | fix redirect_output decorator; remove ninja codeGravatar mathsman51332020-05-16-76/+66
| | | | | | | | | | | | | | | | | | | | | | | | - Lots of instance of `for c in ...` or `for a in ...` or `fmt` which are non-descriptive and sometimes cryptic. - Ves suggested running the command in an asyncio task for `@redirect_output`, rather than making a workaround which only applies to the help command. This fixes a fundamental flaw where the redirection message wouldn't be deleted until a further 60sec after the command has finished, which for `!help` could be up to 5min, meaning the invocation message could be sitting there for 6min, not the intended 60sec.
| * | | Merge branch 'master' into help-refactorGravatar Leon Sandøy2020-05-15-447/+2127
| |\ \ \ | |/ / / |/| | |
* | | | Remove @Admins ping from the #verification messageGravatar Leon Sandøy2020-05-14-1/+1
| | | | | | | | | | | | This probably isn't necessary anymore. We get so many new users that someone is going to DM us very soon when something breaks. We've outgrown this, and it just adds noise to the #verification channel in the form of pings.
* | | | Remove everyone-ping from mentions alertGravatar Sebastiaan Zeeff2020-05-14-1/+0
| | | | | | | | | | | | The mentions alert that is sent out by the Verification cog currently pings `@everyone` despite being quite unactionable by most people receiving the ping. As it happens frequently, especially with the recent uptick in joins, I'm removing that ping to not bother our moderators as much.
* | | | 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