-
-
Notifications
You must be signed in to change notification settings - Fork 89
Add a prune command #330
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
Comments
I agree, purge/prune is a really needed moderation command. Especially for staff assistants, since they don't have a way to delete messages en masse. If a person has permission to manually delete as many messages as they want, they should also be able to do that in a comfortable manner. I also need/want this feature often. For example, to remove past 10 messages in a channel, or to remove past 1h of a user in all channels (something dyno's /purge can't do). It should be flexible enough to cover all use-cases, and I think you have done a good job regarding that. Only issue I have with is UX wise; using the same option for time hour/day and number of messages. Thanks ❤️ |
While it is a duplicate, it probably is better to try it again here. |
@marko-radosavljevic I also agree that using the same option there doesn't feel like the best choice. Do you have anything to propose in it's place that felt practical? |
tl;dr I think we should drop the 'delete last x messages' and go just with 'time'. For example, if there is a spambot, and we want to remove all messages comfortably, we just delete past 2h, and we are done. And if we want to delete a selection, for example, if people have off-topic discussion in #today_you_learn, we can have another command that takes a There are few issues with removing past x messages I can think of:
It doesn't seem very practical UX-wise, and it's dangerous, sooner or later someone will make a mistake. We would need some sort of confirmation button, where the bot would say: Also, we would have to manually count messages, but to do that, we would need to know how exactly to count them. We can achieve a similar result with just time option, and it's easier than counting messages manually. We just check the time unwanted behaviour happened, and then we delete until that point in time. Now, this becomes impractical in case we want to delete deep into the past, and impossible if we want to remove just a selection. So for this functionality we can maybe go with another command, that will take What do you think @Heatmanofurioso? ^^ |
Personally, I have never been in a situation I need a purge command anyways. But I am also a mod, hence I have access to ban+delete message history. Sometimes I have to delete like 4 messages in a row to clear a conversation from a wrong channel; but I would probably continue to do this just with right click -> delete instead of using a command that is likely not flexible enough. So the only real use case that is actually meaningful and not just "because we can", which I personally see here, is that a Staff Helper wants to clear spam because he only has access to kick and per-message-delete. In which case a "delete message history of user X from the past Y time" would be all that is needed. |
^^^ The from (ID) to (ID) is reallyyyyy rare. Takes too much time, doubt anyone will use |
Yeah, it's mostly for staff assistants, since it's super annoying to manually delete spam from every channel. That's also how this issue originated. If's flexible enough, mods can also use it. In case you ever want to delete 20+ messages, it's handy. To clear the channel, to delete messages but without banning the user… We had Shrimp the other day, manually deleting 50 messages, and he is a mod, and we have a dyno's But that is not the primary goal. We want to make simple QoL utility for staff assistants. So yeah, just having time and no additional flags is the cleanest UX, and covers 99% of the cases. An important difference from dyno's |
This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
Would this proposal still be relevant in your eyes @Zabuzard ? Even for another server, it feels like it'd be useful for Staff Helpers against spammers |
We have been talking a lot already on such a feature and what the UX and use cases for it should be. I think we figured out quite well by now what we expect and what not (if reading the history of this chat and the linked chats). So as long as something is created that goes hand in hand with what those discussions resulted in, absolutely go for it. There is definitely a use case for such a command - it is just smaller and more specific than what most people originally proposed. |
Yup, we bounced a lot of ideas about it, it's now up to the person implementing it to frame the best feature out of all this. And we can always improve it further while it's in draft PR phase. I'm always finding new use cases for purge, and I dislike how inflexible discord is with removing messages, and cleaning up channels. For example, I accidentally derailed a suggestion conversation with another member yesterday, and it would be nice to be able to just 'reset it' (with permission of said user), because that serves the most utility for everyone involved. It's very painful when you don't have that flexibility, but you need it, in case of someone spamming, etc. Edit: 15 minutes ago, I removed 6-8 messages from a channel they didn't belong to. Which is not that terrible, since I learned that I can hold shift and delete each message with one click. Before this revelation, it was much more annoying, so this use-case mostly disappeared for me (message count between 6-12 messages), since there wouldn't be a lot of time saved by me typing the command. Still not opposed to it tho, but the primary objective is QoL feature for staff assistants in removing spam from multiple channels. So, personally, I'm open to a lot of different ideas regarding this. Feel free to create a rough draft, and we will see where it goes from there. It's always a good idea to gather as much input before doing any real work, so I appreciate you asking us. ❤️ |
This issue is stale because it has been open 30 days with no activity. Remove stale label, comment or add the valid label or this will be closed in 5 days. |
Add a command to more easily prune spammy user messages
It could have a syntax similar to
/prune {user} {channel/all} {timeframe:1h/1d}
So, for example
/prune @Heatmanofurioso all 1h
-> Would prune all messages sent by the user in the last hour/prune @Heatmanofurioso #lobby 1d
-> Would prune all messages sent by the user in the last day, in the lobby/prune @Heatmanofurioso #lobby 12
-> Would prune the last 12 messages sent by the user in the lobbyWe could also add a parameter for a reason, for logging purposes
The text was updated successfully, but these errors were encountered: