-
Notifications
You must be signed in to change notification settings - Fork 450
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
Allow to clear chat history #1243
Comments
+1 |
The loading speed is down to milliseconds again since we merged #1271 I guess we don't need a workaround like an option to hide the history then? |
Yes, changing the loading order is a good alternative to hiding chat history. |
The chat is now loaded newest to oldest as a temporary workaround. Since emptying all chat might still be a useful feature, eg in webinaries before opening the lobby, I will keep this issue open and update the first post with this information. |
Frontend todos after #5889
|
Reopening for frontend |
@nickvergessen should we also add an occ command to make it possible to automate this ? might need to also provide a timestamp |
I'm not a too big fan of an occ command, as administrators with console access are mostly not the moderators. It's a difference between deploying and terminating a room and moderating it. |
i have a group with multiple users. i send the delete request as user2 but i receive as an answer that user1 cleared the chat:
i would expect either user2 as the actor (or a 403 Forbidden if he is no moderator) |
spreed/tests/integration/features/chat/delete.feature Lines 196 to 209 in b138414
We have an integration confirming exactly this. A non moderator gets a 403 and when a moderator does it both receive a message that says the moderator cleared it. |
ah it seems this is a bug in postman which i use to send requests. seems it doesn't update the auth field correctly after a change... after a restart everything works as expected.. |
Since emptying all chat might still be a useful feature, eg in webinaries before opening the lobby, I will keep this issue open. - @nickvergessen
The text was updated successfully, but these errors were encountered: