bot - A mirrored copy of the python-discord/bot repository.

	Commit message (Collapse)	Author	Age	Lines
...
\| * \| \| \| \| \| \| \|	Merge branch 'master' into feat/backend/b496/rename-constants	S. Co1	2020-02-25	-546/+1676
\| \|\ \ \ \ \ \ \ \
\| * \| \| \| \| \| \| \| \|	Tests: fix setting bot-commands ID in information tests	MarkKoz	2020-02-23	-4/+4
\| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \|	Constants: rename Guild.Constant.ignored to modlog_blacklist	MarkKoz	2020-02-23	-7/+8
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This name better explains what the list is for.
\| * \| \| \| \| \| \| \| \|	Config: replace abbreviated lists with normal ones	MarkKoz	2020-02-23	-4/+34
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Lists were getting too long to be readable as one line. Having each element on a separate line also reduces merge conflicts.
\| * \| \| \| \| \| \| \| \|	Config: add underscore to devcontrib	MarkKoz	2020-02-23	-2/+2
\| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \|	Config: fix DEV_LOG variable thingy	MarkKoz	2020-02-23	-2/+2
\| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \|	Always load doc and verification extensions	MarkKoz	2020-02-23	-5/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	They used to only be loaded in "debug mode" because the main guild was used to test the bot. However, we have since moved to using a separate test guild so it's no longer a concern if these cogs get loaded. It was confusing to some contributors as to why these cogs were not being loaded since the debug mode isn't really documented anywhere.
\| * \| \| \| \| \| \| \| \|	Config: rename channels to match their names in the guild	MarkKoz	2020-02-23	-76/+76
\| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \|	Constants: remove code jam champions role	MarkKoz	2020-02-23	-2/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Nothing was using it.
\| * \| \| \| \| \| \| \| \|	Constants: rename developers role back to verified	MarkKoz	2020-02-23	-1/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	It makes the code which uses it more readable. A comment was added to explain the discrepancy between the constant's name and the name in the guild.
\| * \| \| \| \| \| \| \| \|	Constants: rename roles to match their names in the guild	MarkKoz	2020-02-23	-28/+28
\| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \|	Config: remove checkpoint_test and devtest	MarkKoz	2020-02-23	-6/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	They no longer exist in the guild. * Move devlog under the "Logs" category
\| * \| \| \| \| \| \| \| \|	Config: shorten name of PYTHON_COMMUNITY_ROLE	MarkKoz	2020-02-23	-7/+7
\| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \|	Config: split channels into categories	MarkKoz	2020-02-23	-21/+39
\| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \|	Config: split roles into categories	MarkKoz	2020-02-23	-12/+16
\| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \|	Config: rename roles to match their names in the guild	MarkKoz	2020-02-23	-18/+18
\| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \|	Merge remote-tracking branch 'origin/master' into ↵	MarkKoz	2020-02-25	-1158/+2657
\| \| \| \|\ \ \ \ \ \ \ \| \| \| \|/ / / / / / / \| \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	feat/backend/b131/error-handling
\| \| \| * \| \| \| \| \| \|	Error handler: handle CommandOnCooldown errors	MarkKoz	2020-02-16	-4/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Simply send the error's default message to the invoking context.
\| \| \| * \| \| \| \| \| \|	Error handler: update docstrings to reflect recent changes	MarkKoz	2020-02-16	-20/+39
\| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \|	Error handler: simplify check failure handler & handle bot missing roles	MarkKoz	2020-02-16	-13/+9
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	discord.py's default error messages are quite descriptive already so there really isn't a need to write our own. Therefore, the log calls were removed so that the generic debug log message is used in the on_command_error. In addition to handling missing bot permissions, missing bot roles are also handled. The message doesn't specify which because it doesn't really matter to the end-user. The logs will use the default error messages as described above, and those will contain the specific roles or permissions that are missing.
\| \| \| * \| \| \| \| \| \|	Error handler: (almost) always log the error being handled	MarkKoz	2020-02-16	-16/+11
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The log level is debug for most errors and it's mainly useful for precisely that - debugging. This is why some "useless" errors are also logged e.g. CommandNotFound. Unexpected errors and some API errors will still have higher levels. * Add a single log statement to the end of the handler to cover UserInputError, CheckFailure, and CommandNotFound (when it's not trying to get a tag) * Log 404s from API
\| \| \| * \| \| \| \| \| \|	Error handler: handle ArgumentParsingError	MarkKoz	2020-02-16	-0/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Simply send the error message with the help command.
\| \| \| * \| \| \| \| \| \|	Error handler: handle BadUnionArgument	MarkKoz	2020-02-16	-0/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Send a message specifying the parameter name, the converters used, and the last error message from the converters.
\| \| \| * \| \| \| \| \| \|	Error handler: handle TooManyArguments	MarkKoz	2020-02-16	-0/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Send a message specifying the error reason.
\| \| \| * \| \| \| \| \| \|	Error handler: handle MissingRequiredArgument	MarkKoz	2020-02-16	-1/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Send a message indicating which argument is missing.
\| \| \| * \| \| \| \| \| \|	Error handler: simplify error imports	MarkKoz	2020-02-16	-29/+15
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Import the errors module and qualify the error types with it rather than importing a large list of error types.
\| \| \| * \| \| \| \| \| \|	Error handler: create separate function to handle UserInputError	MarkKoz	2020-02-16	-12/+17
\| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \|	Error handler: move help command retrieval to a separate function	MarkKoz	2020-02-16	-11/+18
\| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \|	Error handler: move tag retrieval to a separate function	MarkKoz	2020-02-16	-18/+28
\| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \|	Error handler: log unhandled exceptions instead of re-raising	MarkKoz	2020-02-16	-2/+2
\| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \|	Error handler: create separate function to handle ResponseCodeError	MarkKoz	2020-02-16	-14/+17
\| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \|	Error handler: create separate function to handle CheckFailure	MarkKoz	2020-02-16	-15/+29
\| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \|	Moderation: fix member not found error not being shown	MarkKoz	2020-02-16	-1/+1
\| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| * \| \|	Merge branch 'master' into fuzzy_zero_div	Sebastiaan Zeeff	2020-02-27	-33/+65
\| \| \| \| \| \| \| \|\ \ \ \| \|_\|_\|_\|_\|_\|_\|/ / / \|/\| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Merge pull request #798 from python-discord/bug/mod/bot-1v/infr-edit-task-cancel	Sebastiaan Zeeff	2020-02-27	-1/+3
\|\ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Moderation: fix task cancellation for permanent infraction when editing
\| * \| \| \| \| \| \| \| \| \|	Moderation: fix task cancellation for permanent infraction when editing	MarkKoz	2020-02-27	-1/+3
\|/ / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	A task should not be cancelled if an infraction is permanent because tasks don't exist for permanent infractions. Fixes BOT-1V
* \| \| \| \| \| \| \| \| \|	Merge pull request #785 from Numerlor/channel_constants_check	Mark	2020-02-27	-29/+53
\|\ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Verify channels against server and reddit OAuth2 in constants on startup.
\| * \ \ \ \ \ \ \ \ \	Merge branch 'master' into channel_constants_check	Mark	2020-02-27	-9/+20
\| \|\ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \|	Merge pull request #790 from ks129/master	S. Co1	2020-02-26	-2/+8
\|\ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Added ignoring staff to Antimalware check.
\| * \| \| \| \| \| \| \| \| \| \|	Add clarifying comment to role checking logic implementation	S. Co1	2020-02-26	-0/+1
\| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \|	Merge branch 'master' into master	S. Co1	2020-02-26	-7/+12
\| \|\ \ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \|	Disable TRACE logging for Sentry breadcrumbs.	Joseph	2020-02-26	-1/+1
\| \|_\|_\|/ / / / / / / / \|/\| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \|	Added DMs ignoring to antimalware check	ks123	2020-02-26	-1/+2
\| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \|	Added to AntiMalware staff ignore check.	Karlis. S	2020-02-26	-1/+5
\| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \|	Fix comparison operator when checking token expiration.	Numerlor	2020-02-26	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	With `<` the check only went through when the token was already expired, making revoking redundant; and didn't go through when the token still had some time before expiration.
\| \| * \| \| \| \| \| \| \| \|	Check for falsy values instead of ``""` and `None` explicitly.	Numerlor	2020-02-26	-2/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	After the change to also check empty strings to avoid unucessary requests, it is no longer necessary to do an explicit value check, as the only values that can come from the .env file are `None` and strings Co-authored-by: Karlis S <[email protected]>
\| \| * \| \| \| \| \| \| \| \|	Specify the logged time is in UTC.	Numerlor	2020-02-26	-1/+1
\| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \|	Create task for `revoke_access_token` when unloading cog to ensure it's ↵	Numerlor	2020-02-26	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	executed.
\| \| * \| \| \| \| \| \| \| \|	Make sure token exists before checking its expiration.	Numerlor	2020-02-26	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Without the check and an invalid token, an AttributeError is raised; blocking the relevant ClientError from being raised in `get_access_token`.
\| \| * \| \| \| \| \| \| \| \|	Check for empty strings alongside None before loading cog.	Numerlor	2020-02-26	-1/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Docker fetches values from the .env itself and defaults to "" instead of None, needing to do invalid access token requests before unloading itself.