Skip to content
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 cut off at 500 letters with no error #71

Open
rejalkarim66666 opened this issue Jun 6, 2014 · 4 comments
Open

Comments cut off at 500 letters with no error #71

rejalkarim66666 opened this issue Jun 6, 2014 · 4 comments

Comments

@rejalkarim66666
Copy link
Contributor

I think it would be better if it either gave an error and/or the letter limit was higher. If not for normal users, then at least for privileged users. Currently it doesn't give any errors and just cuts off everything after 500 letters instead.

@rejalkarim66666 rejalkarim66666 changed the title Comment cuts off above 500 letters Comment cuts off at 500 letters Jun 6, 2014
@rejalkarim66666 rejalkarim66666 changed the title Comment cuts off at 500 letters Comments cut off at 500 letters with no error Jun 6, 2014
@ameliaikeda
Copy link
Contributor

It's a database-level issue. There's nothing at all that checks this but the client-side javascript tracking letter count.

@rejalkarim66666
Copy link
Contributor Author

Normally I'd just condensed it if I hit 500, but I assumed it was different for logged in users or something (it wasn't)

(it should be)

@ameliaikeda
Copy link
Contributor

Database column is varchar(500). I could make it text and limit to 500ish for non-logged-in users.

@9001
Copy link
Contributor

9001 commented Jul 10, 2014

sounds like a good idea, varchar doesnt make much sense for non searchable
text
On Jul 10, 2014 3:23 AM, "Amelia" [email protected] wrote:

Database column is varchar(500). I could make it text and limit to 500ish
for non-logged-in users.


Reply to this email directly or view it on GitHub
#71 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants