-
Notifications
You must be signed in to change notification settings - Fork 231
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 need to be limited in text length #34
Comments
I want to change a bit how comments work in that they should be allowed to be longer and also contain markdown, comments will soon be threaded and resemble a forum. |
Can I just submit a mini pull request closing this issue with a comment limit at 2000 characters? |
After some more thought, deleted my earlier opinion about threaded comments. |
We do need to limit lengths in various ways. First the form field needs to validate size and if it is too large communicate that to user, in the second step the database will reject post sizes above the limit. |
there is now a limit of a few thousand characters - it will be tunable it to whatever one needs |
I was able to post kilobytes of text as comment here: http://biostar.bx.psu.edu/question/show/72/
There should be a server-side limit as well as a JS client side notifier.
The text was updated successfully, but these errors were encountered: