`delete-messages-from-user.js` should remove message instead of clearing them
Especially for users who are banned for spamming, deleting/hiding the messages would be much better than the current behaviour (showing 'This message was deleted').
This fixes almost none of the problems with the spam, as users still have to scroll past walls of text to access the earlier messages. If it's not possible to easily change this for technical reasons, perhaps combining several deleted messages into one could be implemented (e.g. "54 messages were deleted").
Relevant code,
- https://gitlab.com/gitlab-org/gitter/webapp/blob/325f54b85ca69fcade7baa977a44d0d9d2459e8f/scripts/utils/delete-messages-from-user.js#L105-114
- Here is how we delete messages in the UI which does completely remove it, https://gitlab.com/gitlab-org/gitter/webapp/blob/325f54b85ca69fcade7baa977a44d0d9d2459e8f/server/api/v1/rooms/chat-messages.js#L126-127
Edited by Eric Eastwood