| Commit message (Collapse) | Author | Age | Lines |
| ... | |
| | |
|
| |
|
|
| |
This reverts commit 960619c23300c56c8aaa454edc7241e2badf80ad.
|
| |
|
|
| |
All of the tag content is out of scope for this PR.
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| | |
Dynamic views for command help embeds
Adds views for commands to navigate groups.
For subcommands, a button is added to show the parent's help embed.
For groups, buttons are added for each subcommand to show their help embeds.
The views are not generated when help is invoked in the context of an error.
|
| | |
| |
| | |
Migrate to `og_blurple`
|
| | |
| |
| |
| |
| |
| |
| |
| |
| | |
* Run debug log before help command invocation to avoid delayed logs
* Refactored other areas of code slightly, ensuring logging is done as soon as possible. Removed outdated comment
* Ensured debug logs were sent for disabled commands
Co-authored-by: Xithrius <[email protected]>
|
| |\|
| |
| |
| |
| |
| |
| | |
# Conflicts:
# bot/converters.py
# bot/exts/info/tags.py
# tests/bot/test_converters.py
|
| | | |
|
| | |
| |
| |
| | |
This protects us against the guild cache not being fully populated with members.
|
| | |
| |
| |
| |
| | |
The util attaches an error logging callback instead of relying on python's
exception logging which only occurs when the task is destroyed
|
| | | |
|
| |\|
| |
| |
| |
| |
| |
| |
| | |
# Conflicts:
# bot/converters.py
# bot/exts/backend/error_handler.py
# bot/exts/info/source.py
# tests/bot/test_converters.py
|
| | |\ |
|
| | | |\ |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Updates the shh and unshh commands from the error handler to accept
channel and kick arguments, to give them the same interface as the
silence and unsilence command.
Signed-off-by: Hassan Abouelela <[email protected]>
|
| | | |/
| | |
| | |
| | |
| | |
| | |
| | | |
Renames a couple exceptions to include the error suffix, as enforced by
N818.
Signed-off-by: Hassan Abouelela <[email protected]>
|
| | |/ |
|
| | |
| |
| |
| |
| |
| |
| | |
The converter was now only used to restrict requested names which
can be handled by not matching a tag in the cog and not displaying output,
this allows us to be a bit more generous with tag names during the command
fallback when a name with invalid symbols is parsed after a group
|
| | | |
|
| |/
|
|
|
|
| |
The code was restructured to hold tags and their identifiers in
individual classes and some methods moved to function
to detach some of the not directly related functionality from the cog class
|
| |
|
|
|
| |
BadUnionArgument sends command help after:
https://github.com/python-discord/bot/pull/1434
|
| |\ |
|
| | |
| |
| |
| | |
This creates a new baseclass called ContextCheckFailure, and the new error as well as InWhitelistCheckFailure now derive it.
|
| | |
| |
| |
| |
| |
| |
| | |
Instead of:
'January 1 - January 1'
Do:
'January 1'
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Using `datetime.utcnow` produces a tz-naive object. When converting
the object into a POSIX timestamp (L212), the library then converts
the naive object into UTC, which will offset it unless the local
timezone is UTC. We prevent this behaviour by using an Arrow repr
instead, which is by default tz-aware. Since the object already
knows it is in UTC, it does not shift when converting to a timestamp.
Because L233 used `fromtimestamp` rather than `utcfromtimestamp`,
the timestamp then got converted back into local time, canceling
the previous error. Therefore, the bug wasn't observable from logs,
as the times looked correct, but were being stored incorrectly.
By using `Arrow.utcfromtimestamp`, the created object will be aware
of being UTC again, which is more safe.
|
| | |
| |
| |
| |
| | |
Co-authored-by: Shivansh-007 <[email protected]>
Co-authored-by: Joe Banks <[email protected]>
|
| | |
| |
| |
| |
| |
| |
| | |
No code changes in this commit.
Co-authored-by: Shivansh-007 <[email protected]>
Co-authored-by: Joe Banks <[email protected]>
|
| | |
| |
| |
| |
| | |
With the branding-side PR merged, we can now target the production
branch.
|
| | |\
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Conflict in the lockfile resolved by re-locking the merged Pipfile.
Conflict in Branding constants resolved by keeping my local version.
Change in the cog's target branch to 'main' from 'master' is currently
irrelevant as we targets a development branch anyway.
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | | |
It would be strange to just send the embed with no explanation of what
it means.
|
| | | |
| | |
| | |
| | | |
The fallback event should not produce a notification.
|
| | | |
| | |
| | |
| | |
| | |
| | | |
Logs are now proper sentences ended with full stops. Exceptions are
logged with full tracebacks, and log level are revised to be more
sensible and consistent across the extension.
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | | |
No reason for this to be async.
|
| | | |
| | |
| | |
| | |
| | | |
The default KeyError message from dict lookup is just the missing key.
In order to give more context in the log message, we raise our own.
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The fetch helpers will now raise when the request fails rather than
logging a warning and returning a fallback value.
This allows better error logging as the caller is able to log the
propagated exception while adding its own context.
Additionally, the caller in some cases no longer needs to check for
the None return and raise its own exception.
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Previously, the event description & duration strings were only stored
on event entry. In the case that the description or duration change
for an on-going event, the cached values wouldn't be updated.
After this commit, the cache is refreshed daily by the daemon.
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | | |
After previous changes, the docstring was no longer accurate.
See: 1d5625a2f47a1d4d050f9eb0eb7a18e7d6fe171b
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The daemon will now perform a sync iteration immediately when started,
and then every UTC midnight.
Previously, it would only perform the initial iteration when started
for the first time, which is odd.
It is also believed that splitting the daemon's logic into three
separate functions is beneficial: before, loop, and main.
This commit makes log and doc adjustments where appropriate.
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | | |
This should never do anything, but it's better to be safe.
Values taken from Discord developer docs.
|
| | | | |
|
| | | | |
|