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

Add separate Post check for is_valid_body_field #3263

Merged
merged 10 commits into from Jun 26, 2023

Conversation

Neshura87
Copy link
Contributor

Implements #3262

@Neshura87 Neshura87 marked this pull request as ready for review June 22, 2023 10:55
@Neshura87
Copy link
Contributor Author

Neshura87 commented Jun 22, 2023

Woodpecker reran without the commit change, is there any way to rerun the validation?

Edit: Adding another commit by adding an empty line worked

@Nutomic Nutomic merged commit 203e358 into LemmyNet:main Jun 26, 2023
1 check passed
@Neshura87 Neshura87 deleted the patch-1 branch June 26, 2023 08:47
@mpusch88
Copy link
Contributor

mpusch88 commented Jul 4, 2023

It seems like this may not be working. Attempting to post 22417 chars in markdown format results in an 'invalid_body_field' error, and an infinite loading state.

Note that the preview loads fine - I guess it's also possible that there is an illegal character or something.

@Neriderc
Copy link

For anyone arriving here wondering why they can only post 10k characters - well I don't have an answer for you, but know that if you post it first, then you can edit it and it allows the 50k in the edit.

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

Successfully merging this pull request may close these issues.

None yet

4 participants