-
Notifications
You must be signed in to change notification settings - Fork 63
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
Limit message length #186
Comments
Also disable image sharing on the chat box |
Sorry, could you expand on that a bit? We do allow markdown images to be posted into chat, which I think is desireable? |
I just had signed up on the test instance linked to the plugin to check the chat and I could see extremely oversize images being sent in the chat. That could be very heavy on the site or lag the chat a lot depending on the number of people on it. Maybe allow admins to lock the chat per navigated session or allow the chat per to have individual instances per topic/category? Just an idea. |
These are 2 separate issues (but also I agree with @hugoroger... ) |
Currently Babble will allow posts of unlimited size (as far as I can tell) to be posted, Babble totally ignores discourse's max post length setting (default of 32,000 characters), which is still way too much for a shoutbox. A separate setting to limit this - to a much lower value (something like twitter's 140 chars or so.. maybe 200 would be practical), is needed.
The text was updated successfully, but these errors were encountered: