Mark
Mark
> If the bot restarts while the API is down, then tough titties, we'll lose the infractions. Best effort is good enough, given how edge casey this is already. Would...
It may be possible to debug this by having a look through logs. While the logs are significantly less helpful without the trace level, there may still be some errors...
I'm working on reviewing this. Might take me a couple of review sessions.
I've been requested for review, but I do not think all my previous comments have been addressed.
@mbaruh The fetch API call already provides a `User` object with a `name` attribute filled in. Did you mean "nickname" rather than "username"?
Also, can you provide an example of a situation in which you are only able to identify someone by their user/nickname? If it's coming from bot output, then appending the...
> > > @MarkKoz the `UserConverter` relies on the cache, which is usually not enough. Whenever we try look someone up by name#discrim when they've left the guild we get...
Why should it not just be a converter? What advantage is there to invoking it manually?
That sounds like a reasonable implementation to me. What do you think @SebastiaanZ? We can remove the planning label if there's going to be a consensus that 1. this should...
I'm not a fan of the modlog in its current state and welcome improvements to it. I particularly dislike that the cog is used in other cogs. Would you mind...