|  | Commit message (Collapse) | Author | Age | Lines | 
|---|
| ... |  | 
| | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | Reads better as separate lines
Co-Authored-By: Mark <[email protected]> | 
| | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | Co-Authored-By: Mark <[email protected]> | 
| | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | Makes the code a bit clearer
Co-authored-by: Shirayuki Nekomata <[email protected]> | 
| | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | It was triggering an error if the user deleted the output before re-evaluating | 
| | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | This avoid recreating partials for each re-eval | 
| | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | It can be used to test aiohttp request functions, since they are async context managers | 
| | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | The use of lambdas made the functions hard to test, this new format allows us to easily test those functions and document them. | 
| | |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ |  | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | If the eval message is edited after less than 10 seconds, an emoji
is added to the message, and if the user adds the same, the snippet
is re-evaluated. This make easier to correct snipper mistakes. | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | It could lead to a misleading result if it is stripped. | 
| | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | It allows the cog to also work on Windows, because of Signals.SIGKILL not being defined on this platform | 
| | | | | |\ \ \ \ \ \ \ \ \ \ \ \ \ \  
| |_|_|_|/ / / / / / / / / / / / / /  
|/| | | | | | | | | | | | | | | | |   
| | | | | | | | | | | | | | | | | |   
| | | | | | | | | | | | | | | | | | | # Conflicts:
#	bot/cogs/error_handler.py | 
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \  
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | Handle more types of CommandError | 
| | |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \  
| |/ / / / / / / / / / / / / / / / / /  
|/| | | | | | | | | | | | | | | | | | |  | 
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \  
| |_|_|_|_|_|_|_|_|_|_|_|_|_|_|_|/ / /  
|/| | | | | | | | | | | | | | | | | | | Rename constants to match their guild counterparts | 
| | |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \  
| |/ / / / / / / / / / / / / / / / / /  
|/| | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | Delete duplicate keys that were missed in the merge | 
| | |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \  
| | | |_|_|_|_|_|_|_|/ / / / / / / / /  
| | |/| | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | This name better explains what the list is for. | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | Lists were getting too long to be readable as one line. Having each
element on a separate line also reduces merge conflicts. | 
| | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 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. | 
| | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | Nothing was using it. | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 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. | 
| | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | They no longer exist in the guild.
* Move devlog under the "Logs" category | 
| | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | |  | 
| | | | |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \  
| | | |/ / / / / / / / / / / / / / / /  
| | |/| | | | | | | | | | | | | | | |   
| | | | | | | | | | | | | | | | | | | | feat/backend/b131/error-handling | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | Simply send the error's default message to the invoking context. | 
| | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 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. | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | 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 | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | Simply send the error message with the help command. | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | Send a message specifying the parameter name, the converters used, and
the last error message from the converters. | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | Send a message specifying the error reason. | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | Send a message indicating which argument is missing. | 
| | | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | | | | | Import the errors module and qualify the error types with it rather than
importing a large list of error types. | 
| | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | | | | | |  |