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.

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

#855: Ban users without requiring them to be in the server #862

Merged
merged 5 commits into from
Nov 22, 2023

Conversation

butteredwaffles
Copy link
Contributor

Based on some testing I did with a few friends, this should still purge the messages of banned users who left before they were banned.

Copy link

netlify bot commented Nov 6, 2023

Deploy Preview for clembotdocs canceled.

Name Link
🔨 Latest commit 1fa9563
🔍 Latest deploy log https://app.netlify.com/sites/clembotdocs/deploys/65554cc364a4fd000845de55

@butteredwaffles butteredwaffles changed the title #855: Ban users without requiring them to be in the server ClemBotProject/Clembot#855: Ban users without requiring them to be in the server Nov 6, 2023
@butteredwaffles butteredwaffles changed the title ClemBotProject/Clembot#855: Ban users without requiring them to be in the server #855: Ban users without requiring them to be in the server Nov 6, 2023
Copy link

Static type linting failed: Please see CI error output for details and fix the PRs typing

Copy link

Black linting failed: Please run poetry run black bot from the ClemBot.Bot folder

Copy link

Static type linting failed: Please see CI error output for details and fix the PRs typing

Copy link
Member

@Jay-Madden Jay-Madden left a comment

Choose a reason for hiding this comment

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

This is badly needed, some db work might be needed here tho

ClemBot.Bot/bot/services/moderation_service.py Outdated Show resolved Hide resolved
Copy link
Member

@jkriste jkriste left a comment

Choose a reason for hiding this comment

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

LGTM! 🎉

@jkriste jkriste merged commit f8975ab into ClemBotProject:master Nov 22, 2023
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.

3 participants