Skip to content

Chore: Fix the cleanup of Verification channel - #176

Merged
Xarlos89 merged 9 commits into
masterfrom
chore/verification-channel-cleanup
Aug 23, 2026
Merged

Chore: Fix the cleanup of Verification channel#176
Xarlos89 merged 9 commits into
masterfrom
chore/verification-channel-cleanup

Conversation

@kritarth2135

Copy link
Copy Markdown
Collaborator

Description of your PR

Fixes the already written code for the cleanup of verification channel

Describe your changes

Added

None

Fixed

Verification channel cleanup

Changed

Skipping logging deleted messages from the verification process in the LoggingMessageDelete Cog

Removed

None

Issue link

Closes #156

Checklist

  • Does an issue of this PR exist and have you linked it?
  • Have your ran pre-commit over your PR?
  • Was this PR branched off from master?
  • Is this PR merging to master?

@kritarth2135 kritarth2135 self-assigned this Aug 17, 2026
@kritarth2135 kritarth2135 added the chore repo/project maintenance label Aug 17, 2026
@kritarth2135
kritarth2135 requested a review from Xarlos89 August 17, 2026 11:30

@Xarlos89 Xarlos89 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks mostly good, but a few small things that would prevent merging. Nice PR.

Comment thread src/bot/cogs/verification/verification_on_join.py
Comment thread src/bot/cogs/verification/verification_dropdown.py Outdated
Comment thread src/bot/cogs/verification/verification_dropdown.py
@kritarth2135
kritarth2135 force-pushed the chore/verification-channel-cleanup branch from a50522c to ee6a61c Compare August 23, 2026 14:11
@kritarth2135
kritarth2135 requested a review from Xarlos89 August 23, 2026 14:28
@Xarlos89
Xarlos89 merged commit 5b00051 into master Aug 23, 2026
2 checks passed
@Xarlos89
Xarlos89 deleted the chore/verification-channel-cleanup branch August 23, 2026 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore repo/project maintenance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Verification Channel Clog Up

2 participants