Skip to content

Conversation

@tigattack
Copy link
Member

Initial Checklist

  • Has @tigattack been added as a reviewer?
  • If applicable, have the relevant project(s), milestone(s), and label(s) been applied?
  • If applicable, have you added details of the cog to the readme as per README.md?

Details

Does this resolve an issue?
Resolves issue where missing report/emergency command arguments are not handled gracefully or usefully.

Changes

Features / Fixes

  • Fix handling of command errors
    • Actually catch errors from the emergency command
    • Catch missing argument error for report and emergency commands
  • Fix timestamp in cooldown error response to user
  • General improvements around types, linting, etc.

@tigattack tigattack requested a review from rippleFCL September 10, 2025 00:17
@tigattack tigattack added the cog-report Issues related to the report cog. label Sep 10, 2025
@tigattack tigattack merged commit 640d963 into main Sep 10, 2025
5 checks passed
@tigattack tigattack deleted the fix/report_cmds_handle_missing_msg branch September 10, 2025 00:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cog-report Issues related to the report cog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants