| Commit message (Collapse) | Author | Lines |
|
|
|
of points on github
|
|
access the static tag files rather than sending an API get request. Removed all methods calling the API so the tags cannot be edited, added nor deleted.
|
|
|
|
It could have caused some errors if the user delete his own message
|
|
|
|
|
|
A task should not be cancelled if an infraction is permanent because
tasks don't exist for permanent infractions.
Fixes BOT-1V
|
|
Unicode literals aren't really safe compared to code points
|
|
Two functions were created: send_eval and continue_eval, in order to facilitate testing. The corresponding tests are also changed in this commit.
|
|
|
|
|
|
|
|
|
|
|
|
This case is already covered by checking if at least one letter is included.
|
|
The tag fallback didn't convert tags, resulting in possible invalid tag names being passed to the `tags_get_command`. This makes sure they're valid and ignores the risen exception if they are not.
|
|
With only ascii and numbers being allowed to go through, possible values still included things like `$()` which don't match anything in `REGEX_NON_ALPHABET` from tags.py resulting in an error.
|
|
|
|
With `<` the check only went through when the token was already expired, making revoking redundant; and didn't go through when the token still had some time before expiration.
|
|
|
|
After the change to also check empty strings to avoid unucessary requests, it is no longer necessary to do an explicit value check, as the only values that can come from the .env file are `None` and strings
Co-authored-by: Karlis S <[email protected]>
|
|
|
|
executed.
|
|
Without the check and an invalid token, an AttributeError is raised; blocking the relevant ClientError from being raised in `get_access_token`.
|
|
Docker fetches values from the .env itself and defaults to "" instead of None, needing to do invalid access token requests before unloading itself.
|
|
|
|
The previous implementation assumed the config class was a subsection, failing with a KeyError if it wasn't one.
Co-authored-by: kwzrd <[email protected]>
|
|
|
|
Delete duplicate keys that were missed in the merge
|
|
Co-authored-by: SebastiaanZ <[email protected]>
|
|
Co-authored-by: SebastiaanZ <[email protected]>
|
|
When environment variables weren't provided; the cog attempted to create a BasicAuth object with None as values resulting in an exception before the event loop was started and a subsequent crash.
|
|
The function was only used in the since removed `Events` cog.
|
|
|
|
|
|
Adds ConfigVerifier which verifies channels when loaded.
|
|
Python tries to fall back on passing indices to `__getitem__` without iter implemented; failing on the first line.
|
|
* Convert diff namedtuple to dict outside the dict comprehension
* Define long condition as a boolean instead of in the if statement
* Pass role and user dicts to aiohttp normally instead of unpacking
|
|
|
|
This name better explains what the list is for.
|
|
Lists were getting too long to be readable as one line. Having each
element on a separate line also reduces merge conflicts.
|
|
|
|
|
|
They used to only be loaded in "debug mode" because the main guild was
used to test the bot. However, we have since moved to using a separate
test guild so it's no longer a concern if these cogs get loaded.
It was confusing to some contributors as to why these cogs were not
being loaded since the debug mode isn't really documented anywhere.
|
|
|
|
Nothing was using it.
|
|
It makes the code which uses it more readable. A comment was added to
explain the discrepancy between the constant's name and the name in the
guild.
|
|
|
|
They no longer exist in the guild.
* Move devlog under the "Logs" category
|