aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| * | | | | | | | | | | | | | | | | Don't strip whitespaces during snekbox formattingGravatar Matteo Bertucci2020-01-28-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It could lead to a misleading result if it is stripped.
| * | | | | | | | | | | | | | | | | Correct eval output to include the 11th lineGravatar Akarys422019-12-26-1/+1
| | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | Hardcode SIGKILL valueGravatar Akarys422019-12-25-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It allows the cog to also work on Windows, because of Signals.SIGKILL not being defined on this platform
| | | | * | | | | | | | | | | | | | Merge branch 'master' into fuzzy_zero_divGravatar Numerlor2020-02-28-273/+386
| | | | |\ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | # Conflicts: # bot/cogs/error_handler.py
* | | | | | | | | | | | | | | | | | Merge pull request #757 from python-discord/feat/backend/b131/error-handlingGravatar Leon Sandøy2020-02-28-102/+149
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Handle more types of CommandError
| * \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into feat/backend/b131/error-handlingGravatar Leon Sandøy2020-02-28-196/+294
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | | | Merge pull request #782 from python-discord/feat/backend/b496/rename-constantsGravatar Leon Sandøy2020-02-28-163/+229
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|_|_|_|_|_|_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | Rename constants to match their guild counterparts
| * | | | | | | | | | | | | | | | | | Merge branch 'master' into feat/backend/b496/rename-constantsGravatar Leon Sandøy2020-02-28-33/+65
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | Fix mismatched constant names in syncer testsGravatar S. Co12020-02-25-6/+6
| | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | Configure staff role & channel groupings in YAMLGravatar S. Co12020-02-25-10/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Delete duplicate keys that were missed in the merge
| * | | | | | | | | | | | | | | | | | Merge branch 'master' into feat/backend/b496/rename-constantsGravatar S. Co12020-02-25-546/+1676
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | |_|_|_|_|_|/ / / / / / / / / / / | | |/| | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | Tests: fix setting bot-commands ID in information testsGravatar MarkKoz2020-02-23-4/+4
| | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | Constants: rename Guild.Constant.ignored to modlog_blacklistGravatar MarkKoz2020-02-23-7/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This name better explains what the list is for.
| * | | | | | | | | | | | | | | | | | Config: replace abbreviated lists with normal onesGravatar MarkKoz2020-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 devcontribGravatar MarkKoz2020-02-23-2/+2
| | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | Config: fix DEV_LOG variable thingyGravatar MarkKoz2020-02-23-2/+2
| | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | Always load doc and verification extensionsGravatar MarkKoz2020-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 guildGravatar MarkKoz2020-02-23-76/+76
| | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | Constants: remove code jam champions roleGravatar MarkKoz2020-02-23-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Nothing was using it.
| * | | | | | | | | | | | | | | | | | Constants: rename developers role back to verifiedGravatar MarkKoz2020-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 guildGravatar MarkKoz2020-02-23-28/+28
| | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | Config: remove checkpoint_test and devtestGravatar MarkKoz2020-02-23-6/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | They no longer exist in the guild. * Move devlog under the "Logs" category
| * | | | | | | | | | | | | | | | | | Config: shorten name of PYTHON_COMMUNITY_ROLEGravatar MarkKoz2020-02-23-7/+7
| | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | Config: split channels into categoriesGravatar MarkKoz2020-02-23-21/+39
| | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | Config: split roles into categoriesGravatar MarkKoz2020-02-23-12/+16
| | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | Config: rename roles to match their names in the guildGravatar MarkKoz2020-02-23-18/+18
| | |_|_|_|_|_|_|/ / / / / / / / / / | |/| | | | | | | | | | | | | | | |
| | | * | | | | | | | | | | | | | | Merge remote-tracking branch 'origin/master' into ↵Gravatar MarkKoz2020-02-25-1158/+2657
| | | |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | |/ / / / / / / / / / / / / / / | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | feat/backend/b131/error-handling
| | | * | | | | | | | | | | | | | | Error handler: handle CommandOnCooldown errorsGravatar MarkKoz2020-02-16-4/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Simply send the error's default message to the invoking context.
| | | * | | | | | | | | | | | | | | Error handler: update docstrings to reflect recent changesGravatar MarkKoz2020-02-16-20/+39
| | | | | | | | | | | | | | | | | |
| | | * | | | | | | | | | | | | | | Error handler: simplify check failure handler & handle bot missing rolesGravatar MarkKoz2020-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 handledGravatar MarkKoz2020-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 ArgumentParsingErrorGravatar MarkKoz2020-02-16-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Simply send the error message with the help command.
| | | * | | | | | | | | | | | | | | Error handler: handle BadUnionArgumentGravatar MarkKoz2020-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 TooManyArgumentsGravatar MarkKoz2020-02-16-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Send a message specifying the error reason.
| | | * | | | | | | | | | | | | | | Error handler: handle MissingRequiredArgumentGravatar MarkKoz2020-02-16-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Send a message indicating which argument is missing.
| | | * | | | | | | | | | | | | | | Error handler: simplify error importsGravatar MarkKoz2020-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 UserInputErrorGravatar MarkKoz2020-02-16-12/+17
| | | | | | | | | | | | | | | | | |
| | | * | | | | | | | | | | | | | | Error handler: move help command retrieval to a separate functionGravatar MarkKoz2020-02-16-11/+18
| | | | | | | | | | | | | | | | | |
| | | * | | | | | | | | | | | | | | Error handler: move tag retrieval to a separate functionGravatar MarkKoz2020-02-16-18/+28
| | | | | | | | | | | | | | | | | |
| | | * | | | | | | | | | | | | | | Error handler: log unhandled exceptions instead of re-raisingGravatar MarkKoz2020-02-16-2/+2
| | | | | | | | | | | | | | | | | |
| | | * | | | | | | | | | | | | | | Error handler: create separate function to handle ResponseCodeErrorGravatar MarkKoz2020-02-16-14/+17
| | | | | | | | | | | | | | | | | |
| | | * | | | | | | | | | | | | | | Error handler: create separate function to handle CheckFailureGravatar MarkKoz2020-02-16-15/+29
| | | | | | | | | | | | | | | | | |
| | | * | | | | | | | | | | | | | | Moderation: fix member not found error not being shownGravatar MarkKoz2020-02-16-1/+1
| | | | |_|_|_|/ / / / / / / / / / | | | |/| | | | | | | | | | | | |
| | | | | | | * | | | | | | | | | Merge branch 'master' into fuzzy_zero_divGravatar Sebastiaan Zeeff2020-02-27-33/+65
| | | | | | | |\ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|/ / / / / / / / / / |/| | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | Merge pull request #798 from python-discord/bug/mod/bot-1v/infr-edit-task-cancelGravatar Sebastiaan Zeeff2020-02-27-1/+3
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Moderation: fix task cancellation for permanent infraction when editing
| * | | | | | | | | | | | | | | | | Moderation: fix task cancellation for permanent infraction when editingGravatar MarkKoz2020-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_checkGravatar Mark2020-02-27-29/+53
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|_|/ / / / / / / |/| | | | | | | | | | | | | | | | Verify channels against server and reddit OAuth2 in constants on startup.
| * | | | | | | | | | | | | | | | Merge branch 'master' into channel_constants_checkGravatar Mark2020-02-27-9/+20
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | Merge pull request #790 from ks129/masterGravatar S. Co12020-02-26-2/+8
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added ignoring staff to Antimalware check.
| * | | | | | | | | | | | | | | | | Add clarifying comment to role checking logic implementationGravatar S. Co12020-02-26-0/+1
| | | | | | | | | | | | | | | | | |