| Commit message (Collapse) | Author | Age | Lines |
|---|
| ... | |
| |
|
|
|
|
|
|
| |
When you run '!cogs reload *', the load failures will now include
the Exception type, and has overall more readable formatting.
Similarly, trying to do '!cogs load <some cog>' on a broken cog
will show the Exception type as well.
|
| | |
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
While there are measures to prevent duplicate active infractions at
the side of the bot, there's no formal restriction at the sid of the
database. This means that it's possible for a user two get two active
ban infractions or two active mute infractions at the same time, for
instance after a manual table edit in Admin.
This leads to an inconsistent state when unmuting/unbanning that user,
as the ban or mute role will be removed on Discord, but only one of the
entries in the database would be set to inactive, while the other(s)
remain active. This means that an unmuted user will be remuted if they
leave and rejoin the Guild.
To handle this, I've inserted code that sets all the infractions to
inactive and cancels all related infraction tasks (in the case of
temporary infractions) when issuing an unban or unmute command. A note
will be added to the mod_log embed to notify us as well.
|
| |/
|
|
|
|
|
|
|
|
| |
The Deployment cog does not work in our new server environment,
nor are we particularly inclined to make it work (for security
reasons). For this reason, I've removed it.
I've also removed all usages of Roles.devops, replacing them with
Roles.core_developer whenever this made sense to do, such as in
the Cogs cog.
|
| |
|
|
|
| |
This changes URLs for stuff like FAQ, rules, and
the Asking Good Questions page to fit the Django format.
|
| |
|
|
|
|
| |
There was a bug which would permit tag invocations here, because
these were triggered by an error handler on CommandNotFound.
This commit prevents that from being possible.
|
| | |
|
| | |
|
| |\ |
|
| | |\ |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The previous commit assumed the API respone to always be JSON. This
leads to issues when that is not the case, such as when the API is
completely unreachable (text/html 404 response).
Updated the ResponseErrorCode exception to account for that and
updated the moderation util `post_infraction` to reflect that.
|
| | | | |
|
| | | | |
|
| | |/
|/|
| |
| |
| |
| |
| | |
We're not really using this, we're using Roles.verified. This provides
superiour readability, and there's no reason we should use two instead
of just one. I also added a comment to clarify that this role
represents the Developers role on pydis.
|
| |/
|
|
|
|
|
| |
Previously, the regex to detect tokens would only trigger on
messages where the token was inside of single or double quotes.
This commit changes this so that we look for bot tokens regardless
of context.
|
| |
|
|
|
| |
This invokes the `!nomination list` command,
showing all currently nominated users.
|
| |
|
|
|
|
|
|
|
|
|
| |
This commit adds clearer error messages for all the errors
produced by the Wolfram cog if the API key is missing
in the config, or if the key isn't valid anymore.
It also humanizes the timedelta returned in the error
users get when they run out their cooldown. Instead of
telling them they need to wait 84000 seconds, it will now
tell them they need to wait 23 hours, 59 minutes ...
|
| |
|
|
|
|
|
|
| |
The internal 'api' of our API client has changed: It raises a custom
RespondeCodeError instead of an `aiohttp.ClientError` when an API was
not successful. I updated this utility function to catch the right
exception and added handling for unknown users by notifying the user
of that problem directly.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
The ReponseCodeError held a reference to `aiohttp.ResonseObject` to
make sure the response data was available. However, the response data
is not actually included in the Response Object, but needs to be
awaited.
Unfortunately, the ResponseCodeError is usually inspected after the
connection has been closed, triggering a ClientConnectionError when the
data was retrieved.
I solved this by adding the awaited reponse data directly to our custom
exception by awaiting the response.json() before raising the exception.
|
| |
|
|
|
|
|
|
| |
MissingPermissions, CheckFailure, DisabledCommand, and CommandOnCooldown
will now have a simple message logged.
* Log BotMissingPermissions and remove sending missing permissions as a
message
|
| |
|
|
|
|
|
|
|
|
|
| |
https://github.com/python-discord/bot/issues/413
The message indexing phase of the `!clean` did not account for the
presence of the invocation message, resulting in a one-off error in the
amount of messages being indexed. Fixed it by adding one to the amount
of messages we index from the message history.
Closes #413
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Recent changes and updates to the moderation cog introduced some
inconsistencies that were causing bugs or differences in behavior
between very similar commands.
I've remedied the problems by:
- Consistently making sure we stop if a post_infraction API call fails;
- Factoring out the check for active infractions to a utility function;
- Updating commands that expected a pre-migration API response format.
In addition, I've also added function annotations.
|
| |
|
|
|
|
|
|
| |
The bot cog was not updated with recent changes to our
community, so I've:
- Updated the links in the about embed to GitHub;
- Added help-6 and help-7 to the codeblock detection.
|
| |
|
|
|
|
| |
We never added channel IDs for the new help channels
to the constants after adding them, so I'm adding them
in.
|
| |
|
|
|
|
|
|
|
|
|
| |
https://github.com/python-discord/bot/issues/400
The AntiSpam punish method incorrectly invoked the tempmute
command, as it provided an unconverted duration argument. Since
direct invocation of commands bypasses converters, the conversion
of the duration string to a datetime object is now done manually.
Closes #400
|
| |
|
|
|
|
|
|
|
|
| |
https://github.com/python-discord/bot/issues/409
The superstarify cog specified an incorrect infraction type
in the API request in the on_member_join event listener. I've
fixed it by giving it the correct infraction type, 'superstar'.
closes #409
|
| |
|
|
|
|
|
|
|
|
|
|
| |
https://github.com/python-discord/bot/issues/411
The AntiSpam code suffered from a bug where the attribute
self.muted_role was not defined after reloading the cog. The
bug was caused by the cog setting the attribute in on_ready
instead of directly in __init__. Fixed by setting the attribute
in the __init__.
Closes #411
|
| |\
| |
| | |
Update contrib doc for allowing edits from maintainers
|
| | | |
|
| | | |
|
| |/ |
|
| |\
| |
| | |
Updating the deleted message front-end URL to the staff subdomain
|
| | | |
|
| |/ |
|
| |\
| |
| | |
Django role api changes
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| |/ |
|
| |\
| |
| | |
Remove fun cog
|
| | | |
|
| |\ \
| | |
| | | |
Write logs into the site
|
| |/ / |
|
| |\ \
| | |
| | | |
Re-add command error event handler
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | | |
Originally authored by @MarkKoz.
Co-Authored-By: Mark <[email protected]>
|
Leon Sandøy
Sebastiaan Zeeff
kosayoda
MarkKoz
Chris Goes
sco1
Leon Sandøy
Johannes Christ