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.

Sign up for GitHub

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

Jump to bottom

maximum post length setting in acp #2685

Closed
barisusakli opened this issue Feb 3, 2015 · 4 comments
Closed

maximum post length setting in acp #2685

barisusakli opened this issue Feb 3, 2015 · 4 comments
Assignees
@barisusakli
Labels
enhancement request for comments
Milestone
0.6.1

Comments

@barisusakli
Copy link
Member

What is a good default? 5000 chars? :rage1:
@barisusakli barisusakli added this to the 0.6.1 milestone Feb 3, 2015
@barisusakli barisusakli self-assigned this Feb 3, 2015
@MegaGM
Copy link
Contributor

MegaGM commented Feb 3, 2015

32767? Just because

@barisusakli
Copy link
Member Author

Ahhh signed short int

@kevinfmoody
Copy link

@barisusakli :
Just ran into a bug with the following line:

config.minimumPostLength = meta.config.maximumPostLength;

should be

config.maximumPostLength = meta.config.maximumPostLength;

in /src/controllers/api.js.

barisusakli added a commit that referenced this issue Feb 4, 2015
@barisusakli
#2685 fix copy paste fail 😦
fd77380
@barisusakli
Copy link
Member Author

@kevinfmoody thanks for pointing it out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@barisusakli barisusakli

Labels
enhancement request for comments
Projects
None yet
Milestone
0.6.1
Development

No branches or pull requests
3 participants
@kevinfmoody @barisusakli @MegaGM