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

/quarantine command should also give option to remove message history #1064

Open
ankitsmt211 opened this issue Mar 20, 2024 · 2 comments
Open
Labels
enhancement New feature or request priority: normal

Comments

@ankitsmt211
Copy link
Member

Is your feature request related to a problem? Please describe.

Currently if you quarantine a spam bot or an individual with compromised account, it does not purges those scam links. You have to delete em by visiting each channel one at a time, you might even miss some of those.

Describe the solution you'd like

/ban takes an option whether you wanna clear users message history, something like that could also be plugged into /quarantine

Additional context

Ban Command

Quarantine command

@ankitsmt211 ankitsmt211 added enhancement New feature or request priority: normal labels Mar 20, 2024
@Zabuzard
Copy link
Member

Removing message history is difficult without running into rate limits.

Its doable, but similar to Cake Day, requires to spread out all operations over some time.

The problems start with not being able to retrieve a message history of a user in the first place. Only per-channel. (Although, the use case this feature is needed for doesnt require a full and proper message history, its enough to peek the last 10 messages in all channels). And then individual deletion of all these messages, as I dont think they can be bulk-deleted if from different channels.

Requires some API investigation, but probably not that simple.

An alternative approach could be to have the command ban and then unban the user (as the ban command has discord built-in history deletion support), while explaining what happened to the user in DM and that they can join back.

@ankitsmt211
Copy link
Member Author

I wasn't aware /ban was inbuilt feature from discord, in that case #982 might be able to handle this in future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority: normal
Projects
None yet
Development

No branches or pull requests

2 participants