Skip to content

Update MAX_COMMENT_LENGTH to 15k to match with BE limitation #69653

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

Merged
iwiznia merged 4 commits into from
Sep 4, 2025
Merged

Update MAX_COMMENT_LENGTH to 15k to match with BE limitation #69653

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
a0b63c7
fix: match the message limit with BE
lorretheboy Sep 2, 2025
a643a86
chore: rerun workflow
lorretheboy Sep 2, 2025
124e13f
chore: rerun workflow
lorretheboy Sep 2, 2025
b63e59e
Merge branch 'main' of https://github.com/Expensify/App into fix/68933
lorretheboy Sep 4, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/CONST/index.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3711,7 +3711,7 @@ const CONST = {
},

// Auth limit is 60k for the column but we store edits and other metadata along the html so let's use a lower limit to accommodate for it.
MAX_COMMENT_LENGTH: 10000,
MAX_COMMENT_LENGTH: 15000,

// Use the same value as MAX_COMMENT_LENGTH to ensure the entire comment is parsed. Note that applying markup is very resource-consuming.
MAX_MARKUP_LENGTH: 10000,
Expand Down
Loading