Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Improve report cog #309

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Improve report cog #309

wants to merge 11 commits into from
+157 −124

Conversation

tigattack
Copy link
Member

@tigattack tigattack commented Feb 21, 2025 •
edited
Loading

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 #282
Resolves #42

Changes

Features / Fixes

  • Implements slash commands for report and emergency.
  • Adds link to message before report in report embed.
  • Adds [p]reports status command.
  • Adds 1/30s cooldown for report and emergency commands (text-based and slash).
  • DM bot owner(s) when a report is made without a configured log channel.

Breaking Changes

  • Removes enabled_channels functionality. The same can be achieved with Red's permissions functionality.
  • The [p]reports confirm command is now [p]reports confirmation.
  • The [p]reports logchannel and [p]reports confirmation commands can only be run from within a guild.

@tigattack
refactor(report): improve config checks & responses
c24fe6e
@tigattack
refactor(report): extract report logic to method
7ed73e0 
* Stop throwing `ctx` around to keep helper methods a little 'tighter'.
* Add `get_log_channel` helper method to reduce `do_report` method complexity.
* Restructure report confirmation embed.
@tigattack
feat(report)!: remove enabled_channels functionality
c74a0c7 
This can be achieved with Red's [`permissions`](https://docs.discord.red/en/stable/cog_guides/permissions.html) functionality.
@tigattack
feat(report): make report and emergency hybrid commands
f4331d1 
Ref. https://docs.discord.red/en/stable/guide_slash_and_interactions.html#hybrid-commands
@tigattack
feat(report): rework report & confirmation embeds
c0c0a93 
Main feature is adding a "Context Region" field which links to the message directly before the user's report.
@tigattack
fix(reports): make logchannel and confirm cmds guild_only
7ca958b
@tigattack
feat(report): add cooldown to report and emergency cmds
e1f3b63
@tigattack
feat(report): add status cmd
8a601d7
@tigattack
refactor(report): rename confirm cmd to confirmation
6741491
@tigattack
docs(report): update usage info, document new features
427dfde
@tigattack
docs(report): add cooldown info
1f35fce
@tigattack tigattack added the cog-report Issues related to the report cog. label Feb 21, 2025
@tigattack tigattack marked this pull request as ready for review February 21, 2025 22:37
@tigattack tigattack changed the title refactor(report): improve config checks & responses Improve report cog Feb 22, 2025
@tigattack tigattack requested a review from rippleFCL February 22, 2025 00:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rippleFCL rippleFCL Awaiting requested review from rippleFCL

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
cog-report Issues related to the report cog.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[FEAT] report command as slash command [FEAT] Link to message before trigger in report cog.
1 participant
@tigattack