TizzySaurus

Results 61 comments of TizzySaurus

> Bug 1. I don't know if it's worth it to do a second API call, when the current behavior doesn't seem to be a problem. I don't mind either...

After further discussion on discord, I've decided that Bug 1 is fine as-is, and not really a big deal. The two enhancements have been implemented in #2232, and Bug 2...

> Relative action time in footer (per @MarkKoz here) I discussed this with one of the core devs (I believe in a mod channel) and we decided we don't want...

> Although, in practice the action time shouldn't be too distant from the report time, so the information is technically in the embed already. I mean there are multiple instances...

This latest commit is untested as I'm on holiday, so have added the "do not merge" tag to make sure it doesn't get merged before confirming it all works as...

Have tested and everything works as expected: ![image](https://user-images.githubusercontent.com/47674925/180595102-899b2749-5036-4eb1-8c60-e37bb2eb0292.png) As such, this is now ready for review :smile:

> LGTM. Seems like there are some errors in the tests that need fixing though Turns out I forgot to push the commit where I fixed the tests 😅 Have...

> Is this still necessary? I think the issue was fixed in [2b2dc95](https://github.com/python-discord/bot/commit/2b2dc95f2aa2ac6606b51be183861f2663be3b3e) Without actually testing anything, it does indeed look like this issue has been fixed (which would mean...

> Hey @TizzySaurus (for your PR description) the tempfix was for `.hackstats` not `.hacktoberissues` :) Fixed, thanks :+1:

> @TizzySaurus is this PR only focused on migrating the code as-is and the restructuring or are you wanting to refactor the three commands as well? Many of Jason's suggestions...