| Commit message (Collapse) | Author | Lines |
|
We're moving the actual predicate into the `utils.checks` folder, just
like we're doing with most of the other decorators. This is to allow us
the flexibility to use it as a pure check, not only as a decorator.
This commit doesn't actually change any functionality, just moves it
around.
|
|
|
|
|
|
At this point, it's just a thin wrapper to call another function. It's
redundant.
|
|
It was being done repeatedly outside the function so let's move it in
to reduce redundancy.
|
|
|
|
|
|
Co-authored-by: Mark <[email protected]>
|
|
- Created function `embed_description_match`.
- Implemented this to `is_empty`
- Implemented this to `is_dormant_message`
|
|
Co-authored-by: Mark <[email protected]>
|
|
- Created function `is_empty` that check is there any message in channel
after bot own available message.
- `on_message_delete` that reschedule task when message is on correct
channel and is empty.
- In `move_idle_channel` function, implemented choosing right cooldown,
based on is channel empty or not.
|
|
This show how much minutes should this wait before making channel
dormant when no messages in channel (original message deleted).
|
|
Co-authored-by: Joseph Banks <[email protected]>
|
|
Co-authored-by: Joseph Banks <[email protected]>
|
|
|
|
publish can take place
|
|
|
|
|
|
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)`
|
|
|
|
|
|
Co-authored-by: kwzrd <[email protected]>
|
|
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.
|
|
|
|
|
|
|
|
|
|
|
|
Collect Guild boost amount + level and post it to StatsD every hour in task. Added starting to cog `__init__.py` and stopping to `cog_unload`.
|
|
- 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.
|
|
|
|
|
|
|
|
Add stat increaser to PEP and maillist posting.
|
|
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.
|
|
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.
|
|
|
|
|
|
Another refactor/cleaning to make the logic clearer and easier to understand. Also cleaned up the trace logs to be shorter and more concise. Thanks, @scragly!
Co-authored-by: scragly <[email protected]>
|
|
|
|
Invalid invites won't have data available to put in the embeds.
Fixes #929
Fixes BOT-3Z
|
|
Bots are incapable of deleting direct messages authored by others.
|
|
- Refined the logic for `apply_ban()` even further to be cleaner. (Thanks, @MarkKoz!)
Signed-off-by: Daniel Brown <[email protected]>
|
|
- Changed docstring explanation and function name of `get_active_infractions` to `get_active_infraction()` to better convey that only one infraction is returned. Also changed all relevant uses to reflect that change.
- Added explanation of parameter `send_msg` to the doc strings of `pardon_infraction()` and `get_active_infraction()`
- Adjusted placement of `log.trace()` in `pardon_infraction()`
- Adjusted logic in `apply_ban()` to remove redundant check.
- Adjusted logic in `apply_ban()` to be consistent with other checks.
Signed-off-by: Daniel Brown <[email protected]>
|
|
Co-authored-by: Joseph Banks <[email protected]>
|
|
|
|
confusion
|
|
Co-authored-by: Sebastiaan Zeeff <[email protected]>
|
|
than 2000 chars
|
|
- Changed `has_active_infraction` to `get_active_infractions` in order to add additional logic in `apply_ban`.
- Added `send_msg` parameters to `pardon_infraction` and `get_active_infractions` so that multi-step checks and actions don't need to send additional messages unless told to do so.
Signed-off-by: Daniel Brown <[email protected]>
|