bot
bot copied to clipboard
python-discord
Incident archive msg improvements
Closes #1059
Incident archive messages now show when the incident was made and no longer displays when the incident was actioned as it's the same as the message timestamp.
Also removed some redundant escapes in the discord message link regex.
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 this statistic, as it's often not particularly accurate (people forgetting to action, or an incident taking a long time to resolve due to a ModMail thread etc.) and causes extra pressure on Moderators to action incidents sooner, which isn't wanted.
Absolute action time as footer timestamp
For this I refer back to this PR's description: "... and no longer displays when the incident was actioned as it's the same as the incident-archive message timestamp." This isn't needed.
EDIT: I suppose there is a case here; when multiple reports by the same user are actioned. In this scenario, only the first will have the timestamp. You can know it was actioned within a few minutes of the previous; is this accurate enough or do we want to re-add the timestamp for precision?
Simplified time of report (less clutter, straight to the point)
I do agree with this point; it definitely looks much nicer so will change.
we decided we don't want this statistic
Sure, makes sense.
when multiple reports by the same user are actioned.
In compact mode, if I want to see the date an incident was actioned, I would have to scroll to the first incident of that day, so adding the timestamp to the footer helps. Although, in practice the action time shouldn't be too distant from the report time, so the information is technically in the embed already.
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 where it's several hours after, simply because we're unfortunately not a 24/7 taskforce. Then there's the odd case of it being days later (due to being delayed by a ModMail thread etc.).
I think supported with your compact mode comment, that's enough rationale for re-adding the timestamp.
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 planned.
Have tested and everything works as expected:
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 done so now :+1: