Skip to content

Implement timestamp to reported messages#763

Merged
MattyTheHacker merged 2 commits intomainfrom
add-time-to-report
May 6, 2026
Merged

Implement timestamp to reported messages#763
MattyTheHacker merged 2 commits intomainfrom
add-time-to-report

Conversation

@MattyTheHacker
Copy link
Copy Markdown
Member

No description provided.

@MattyTheHacker MattyTheHacker self-assigned this May 6, 2026
Copilot AI review requested due to automatic review settings May 6, 2026 13:33
@MattyTheHacker MattyTheHacker added enhancement New feature or request sync Request bots to automatically keep this PR up to date with it's base branch labels May 6, 2026
@MattyTheHacker MattyTheHacker enabled auto-merge (squash) May 6, 2026 13:34
@codecov
Copy link
Copy Markdown

codecov Bot commented May 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds the original message’s creation time to the committee report embed so moderators can immediately see when the reported content was posted.

Changes:

  • Include message.created_at as the embed timestamp when forwarding a reported message to the committee channel.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Member

@CarrotManMatt CarrotManMatt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you send an image of how this timestamp shows up in the UI of the message please?

@MattyTheHacker
Copy link
Copy Markdown
Member Author

MattyTheHacker commented May 6, 2026

Could you send an image of how this timestamp shows up in the UI of the message please?

image

jk will do later

@MattyTheHacker
Copy link
Copy Markdown
Member Author

MattyTheHacker commented May 6, 2026

image @CarrotManMatt

@MattyTheHacker MattyTheHacker merged commit bc210f4 into main May 6, 2026
13 checks passed
@MattyTheHacker MattyTheHacker deleted the add-time-to-report branch May 6, 2026 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request sync Request bots to automatically keep this PR up to date with it's base branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants