aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| | | * | | HelpChannels: add a function to get a channel or fetch it from APIGravatar MarkKoz2020-03-22-0/+8
| | | | | |
| | | * | | HelpChannels: add a loggerGravatar MarkKoz2020-03-22-0/+3
| | | | | |
| | | * | | HelpChannels: add method stubsGravatar MarkKoz2020-03-22-1/+48
| | | | | |
| | | * | | Constants: add help category constantsGravatar MarkKoz2020-03-22-3/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The original category was re-purposed as the "in-use" category so that deployment of the new system will not interrupt ongoing help sessions.
| | | * | | HelpChannels: add constants for active/dormant messagesGravatar MarkKoz2020-03-22-0/+24
| | | | | |
| | | * | | Constants: add constants for HelpChannels cogGravatar MarkKoz2020-03-22-0/+19
| | | | | |
| | | * | | HelpChannels: load element names from JSONGravatar MarkKoz2020-03-22-0/+7
| | | | | |
| | | * | | HelpChannels: create boilerplate extension and cogGravatar MarkKoz2020-03-22-0/+12
| | | | | |
| | | * | | Resources: add JSON with array of chemical element namesGravatar MarkKoz2020-03-22-0/+120
| | | | | |
| | | | * | 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
| |_|_|_|_|_|/ |/| | | | | |
* | | | | | | Merge pull request #852 from ks129/infraction-editGravatar S. Co12020-03-31-1/+6
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Don't let change expiration time when infraction already expired.
| * \ \ \ \ \ \ Merge branch 'master' into infraction-editGravatar S. Co12020-03-31-60/+562
| |\ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | |
* | | | | | | | Merge pull request #854 from ks129/tags-trashcan-reactGravatar S. Co12020-03-31-5/+17
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Tags response + command message deletion with trashcan reaction
| * \ \ \ \ \ \ \ Merge branch 'master' into tags-trashcan-reactGravatar S. Co12020-03-31-55/+545
| |\ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | |
* | | | | | | | | Merge pull request #846 from python-discord/feat/filter/322/merge-tokens-wordsGravatar S. Co12020-03-31-41/+21
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Filtering: merge the word and token watch filters
| * \ \ \ \ \ \ \ \ Merge branch 'master' into feat/filter/322/merge-tokens-wordsGravatar S. Co12020-03-31-15/+617
| |\ \ \ \ \ \ \ \ \ | |/ / / / / / / / / |/| | | | | | | | |
* | | | | | | | | | Merge pull request #833 from ks129/zen-match-fixGravatar Sebastiaan Zeeff2020-03-31-1/+11
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Fixed !zen command exact word matching.
| * \ \ \ \ \ \ \ \ \ Merge branch 'master' into zen-match-fixGravatar Karlis S2020-03-31-14/+606
| |\ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / |/| | | | | | | | | |
* | | | | | | | | | | Merge pull request #845 from python-discord/update-logging-levelsGravatar Leon Sandøy2020-03-31-10/+9
|\ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|_|/ |/| | | | | | | | | | Logging Level Adjustments
| * | | | | | | | | | Merge branch 'master' into update-logging-levelsGravatar Leon Sandøy2020-03-31-12/+656
| |\ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / |/| | | | | | | | | |
* | | | | | | | | | | Set unsilence permissions to inherit instead of trueGravatar Sebastiaan Zeeff2020-03-30-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The "unsilence" action of the silence/hush command used `send_messages=True` when unsilencing a hushed channel. This had the side effect of also enabling send messages permissions for those with the Muted rule, as an explicit True permission apparently overwrites an explicit False permission, even if the latter was set for a higher top-role. The solution is to revert back to the `Inherit` permission by assigning `None`. This is what we normally use when Developers are allowed to send messages to a channel.
* | | | | | | | | | | Merge pull request #812 from Numerlor/hush-cogGravatar Eivind Teig2020-03-30-3/+504
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Implement silencer (hush) cog.
| * \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into hush-cogGravatar Eivind Teig2020-03-30-548/+1802
| |\ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / |/| | | | | | | | | | |
| * | | | | | | | | | | Add tests for `HushDurationConverter`.Gravatar Numerlor2020-03-17-0/+30
| | | | | | | | | | | |
| * | | | | | | | | | | Add tests ensuring permissions get preserved.Gravatar Numerlor2020-03-17-0/+31
| | | | | | | | | | | |
| * | | | | | | | | | | Use set `discard` instead of `remove`.Gravatar Numerlor2020-03-17-3/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Discard ignores non present values, allowing us to skip the KeyError suppress.
| * | | | | | | | | | | Add docstring to test.Gravatar Numerlor2020-03-17-0/+1
| | | | | | | | | | | |
| * | | | | | | | | | | Fix test name.Gravatar Numerlor2020-03-17-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `removed` was describing the opposite behaviour.