| Commit message (Collapse) | Author | Age | Lines |
|
|
| |
Co-authored-by: Mark <[email protected]>
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
| |
Replace `{"foo": "bar"}` with `{"id": 1}`
|
|
|
| |
To allow `.get`, I had to replace `str` return value with `dict`
|
| |
|
| |
|
| |
|
| |
|
|\ |
|
| |\
| | |
| | | |
Increase snekbox re eval timeout to 30 seconds
|
| | |\
| | |/
| |/| |
|
| | | |
|
| | |
| | |
| | |
| | | |
publish can take place
|
| |\ \
| | | |
| | | | |
Add message publishing to Reddit cog
|
| | |\ \
| | |/ /
| |/| | |
|
| |\ \ \
| | | | |
| | | | | |
Add mutability.md tag
|
| | |\ \ \
| | |/ / /
| |/| | | |
|
| |\ \ \ \
| | | | | |
| | | | | | |
Use `send_help` to ensure that our help command is correctly invoked
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
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)`
|
| |/ / / /
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
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.
|
| |\ \ \ \
| | | | | |
| | | | | | |
Refactor the !help command.
|
| | |\ \ \ \
| | |/ / / /
| |/| | | | |
|
| |\ \ \ \ \
| | | | | | |
| | | | | | | |
add "solved" as a alias for "closed"
|
| |/ / / / / |
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
- 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.
|
| | |\ \ \ \
| | |/ / / /
| |/| | | | |
|
| | | | | | |
|
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Not sure why my precommit didn't pick that up...
|
| | |\ \ \ \
| | | | | | |
| | | | | | |
| | | | | | | |
help-refactor
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
- Make exception handling for bin reaction more specific
- Channel constants were updated recently
- Suggest category names
- Tidy up signature formatting
- Move score cutoff to 80 to allow a few more matches
|
| | |\ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
help-refactor
Conflicts:
bot/cogs/help.py
|
| | | | | | | | |
|
| | | | | | | | |
|
| | |\ \ \ \ \ \
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
help-refactor
Conflicts:
bot/cogs/help.py
|
| | | | | | | | | |
|
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
- Description was the same as prefix parameter of paginator
- Cleanup is redundant pending closure of #514
- Clean/fix couple if statements in help.py
|
| | | | | | | | | |
|
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
- Change `add_field` back to `description` for error message possible matches
- Only add `Commands` and `Subcommands` if subcommands exist to cog/group/command help
|
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
- Before, running `!help` would invoke the cooldown check, and increase the cooldown counter unnecessarily as no wolfram API calls were being made.
- Once `!help` was called enough, the bot would send an error embed to let you know your wolfram cooldown has expired.
|
| | | | | | | | | |
|
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
- `redirect_output` has been adjusted to run the `delete_invocation` inside a task as the help command will wait for that to run before sending the help or doing anything else.
- `pagination` has been adjusted to support deleting the paginated message if `cleanup` is True, and an optional `description` that is present through all pages of pagination.
- The help command has been refactored to subclass `commands.HelpCommand`. This means that it now supports methods such as `ctx.send_help(ctx.command)`.
- `help_cleanup` provides the opportunity to use the :x: reaction to cleanup help even with no pagination.
- Pagination purely happens through the `LinePaginator`, forcing a pagination session with 1 line per page where we format the page style before sending it through.
- Categories are properly dealt with by finding a match and sending a seperate help where a named tuple of the Category name, description and relevant cogs is the only parameter.
- Choices for when a command was not found has been updated to include category names, cog names, aliases of group and command names, and include all subcommands and aliases. This should provide a more helpful output when an error message is sent
- Sending command, group, cog, category and bot help has been split into different functions that are called from `command_callback`. This provides an easier way to alter future changes, and cleans up code considerably.
- Important note: no outward facing formatting should have changed. Any desired changes can be discussed in review.
|
| | | | | | | | | |
|
| | | | | | | | | |
|
| | | | | | | | | |
|
| | | | | | | | | |
|