Skip to content

fix: delete messages on ban and kick#86

Merged
toto04 merged 2 commits intomainfrom
ban-delete
Apr 11, 2026
Merged

fix: delete messages on ban and kick#86
toto04 merged 2 commits intomainfrom
ban-delete

Conversation

@lorenzocorallo
Copy link
Copy Markdown
Member

No description provided.

@lorenzocorallo lorenzocorallo requested a review from toto04 April 11, 2026 16:10
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 11, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 765ad4f8-65d9-4641-a6c3-152f503e8d7d

📥 Commits

Reviewing files that changed from the base of the PR and between 6f9b761 and 24f5725.

📒 Files selected for processing (1)
  • src/modules/moderation/index.ts

Walkthrough

Updated the moderation module to include message revocation functionality. The changes add revoke_messages: true parameter to the banChatMember API payloads for KICK and BAN actions, enabling deletion of messages when users are kicked or banned.

Changes

Cohort / File(s) Summary
Moderation Message Revocation
src/modules/moderation/index.ts
Added revoke_messages: true to banChatMember API calls for KICK and BAN actions to enable automatic message deletion during moderation operations.
🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'fix: delete messages on ban and kick' directly and accurately describes the main change: adding message deletion functionality to ban and kick actions in the moderation module.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@toto04 toto04 merged commit aacb658 into main Apr 11, 2026
2 checks passed
@toto04 toto04 deleted the ban-delete branch April 11, 2026 17:06
This was referenced Apr 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants