aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* ModLog: Fix embed description truncationGravatar ks1292020-05-20-1/+1
|
* Infractions: Remove space from placeholderGravatar ks1292020-05-20-1/+1
|
* Merge branch 'master' into ban-kick-reason-lengthGravatar ks1292020-05-20-870/+2369
|\
| * Merge pull request #944 from Numerlor/eval-timeout-increaseGravatar Dennis Pham2020-05-19-2/+7
| |\ | | | | | | Increase snekbox re eval timeout to 30 seconds
| | * Merge branch 'master' into eval-timeout-increaseGravatar Dennis Pham2020-05-19-503/+349
| | |\ | | |/ | |/|
| * | 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
| | |\ \ \ \ | | |/ / / / | |/| | | |
| | * | | | Help: lower score cutoff for fuzzy matchGravatar Mark2020-04-01-1/+1
| | | | | |
| | * | | | Fix linting?Gravatar mathsman51332020-03-31-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Not sure why my precommit didn't pick that up...
| | * | | | Merge branch 'master' of https://github.com/python-discord/bot into ↵Gravatar mathsman51332020-03-31-375/+1918
| | |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | help-refactor
| | * | | | | Apply suggestions from review.Gravatar mathsman51332020-03-31-12/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - 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
| | * | | | | Merge branch 'master' of https://github.com/python-discord/bot into ↵Gravatar mathsman51332020-03-12-2207/+4239
| | |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | help-refactor  Conflicts:  bot/cogs/help.py
| | * | | | | | Apply suggestions from Mark's code review.Gravatar mathsman51332020-03-10-60/+63
| | | | | | | |
| | * | | | | | Use the new :trashcan: emoji to delete the help message, as per #625Gravatar mathsman51332020-02-10-4/+5
| | | | | | | |
| | * | | | | | Merge branch 'master' of https://github.com/python-discord/bot into ↵Gravatar mathsman51332020-02-10-3174/+7258
| | |\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | help-refactor  Conflicts:  bot/cogs/help.py
| | * | | | | | | Remove trailing commaGravatar mathsman51332019-11-16-1/+1
| | | | | | | | |
| | * | | | | | | Apply suggestions from reviewGravatar mathsman51332019-11-16-24/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Description was the same as prefix parameter of paginator - Cleanup is redundant pending closure of #514 - Clean/fix couple if statements in help.py
| | * | | | | | | Show a maximum of 8 commands per page rather than 5.Gravatar mathsman51332019-11-03-1/+1
| | | | | | | | |
| | * | | | | | | Few changes to keep formatting same as currentGravatar mathsman51332019-10-13-5/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Change `add_field` back to `description` for error message possible matches - Only add `Commands` and `Subcommands` if subcommands exist to cog/group/command help
| | * | | | | | | Add a special case for when the help command invokes wolfram checks.Gravatar mathsman51332019-10-13-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - 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.
| | * | | | | | | Minor formatting changes to align with current help.Gravatar mathsman51332019-10-12-2/+2
| | | | | | | | |
| | * | | | | | | Refactor the `Help` command.Gravatar mathsman51332019-10-12-475/+287
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - `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.
| | | | | | * | | Fix incomplete variable renamingGravatar decorator-factory2020-05-17-1/+1
| | | | | | | | |
| | | | | | * | | Rename `string` to `greeting`Gravatar decorator-factory2020-05-17-10/+10
| | | | | | | | |
| | | | | | * | | Add a note on user-defined classesGravatar decorator-factory2020-05-17-1/+1
| | | | | | | | |
| | | | | | * | | Change standalone programs to interactive sessionsGravatar decorator-factory2020-05-17-8/+13
| | | | | | | | |
| | | | | | * | | Apply language improvements proposed from kwzrdGravatar decorator-factory2020-05-17-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: kwzrd <[email protected]>
| | | | | | * | | Fix hard-wrapping in mutability.mdGravatar decorator-factory2020-04-01-7/+5
| | | | | | | | |
| | | | | | * | | header->bold in mutability.mdGravatar decorator-factory2020-04-01-1/+1
| | | | | | | | |
| | | | | | * | | Add mutability.md tagGravatar decorator-factory2020-04-01-0/+34
| | | | | | | | |
| | | | | | | * | Add message publishing to `Reddit` cogGravatar ks1292020-05-13-1/+7
| | | | | | | | |
| | | | | | | | * Change tests to use the new timeout constantGravatar Numerlor2020-05-15-1/+5
| | | | | | | | |
| | | | | | | | * Move the re eval timeout to a module constantGravatar Numerlor2020-05-15-1/+2
| | | | | | | | |
| | | | | | | | * Increase snekbox re eval timeout.Gravatar Numerlor2020-05-15-1/+1
| | |_|_|_|_|_|/ | |/| | | | | |
| * | | | | | | 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