[PR #30156] fix: remove hardcoded 48-character limit from text inputs #32708

Closed
opened 2026-02-21 20:51:55 -05:00 by yindo · 0 comments
Owner

Original Pull Request: https://github.com/langgenius/dify/pull/30156

State: closed
Merged: Yes


Summary

  • Removes the hardcoded DEFAULT_VALUE_MAX_LEN (48 characters) fallback from text input components
  • When max_length is not configured, inputs now have no character limit as expected
  • Only applies maxLength when explicitly configured by the user

Test plan

  • All existing tests pass (5458 tests)
  • Added new tests to verify maxLength behavior:
    • Input should not have maxLength attribute when max_length is not set
    • Input should have maxLength attribute when max_length is set

fixes #26940

**Original Pull Request:** https://github.com/langgenius/dify/pull/30156 **State:** closed **Merged:** Yes --- ## Summary - Removes the hardcoded `DEFAULT_VALUE_MAX_LEN` (48 characters) fallback from text input components - When `max_length` is not configured, inputs now have no character limit as expected - Only applies `maxLength` when explicitly configured by the user ## Test plan - [x] All existing tests pass (5458 tests) - [x] Added new tests to verify maxLength behavior: - Input should not have maxLength attribute when max_length is not set - Input should have maxLength attribute when max_length is set fixes #26940
yindo added the pull-request label 2026-02-21 20:51:55 -05:00
yindo closed this issue 2026-02-21 20:51:55 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langgenius/dify#32708