- Issue created by @omkar.podey
- ๐ฎ๐ณIndia omkar.podey
Since there is no defined upper limit it makes sense to just add a lower limit for the default value and that it's not null I think.
- Status changed to Needs review
9 months ago 10:25am 27 March 2024 - Status changed to Needs work
9 months ago 4:05pm 27 March 2024 - ๐ฎ๐ณIndia narendraR Jaipur, India
smustgrave โ credited narendraR โ .
- ๐บ๐ธUnited States smustgrave
Appears to be feedback from @NarendraR, adding credit for the MR review.
- Status changed to Needs review
9 months ago 5:34am 28 March 2024 - Status changed to RTBC
9 months ago 8:23am 28 March 2024 - Status changed to Fixed
9 months ago 9:45am 28 March 2024 - ๐ฌ๐งUnited Kingdom alexpott ๐ช๐บ๐
I think this configuration is pointless - or should be used in far more places. For example: \Drupal\text\Plugin\Field\FieldFormatter\TextTrimmedFormatter::defaultSettings() but even then it wouldn't change after the of the field being set up. Can we file a follow-up to discuss removing it?
Committed and pushed 9555002a53 to 11.x and e140d4a00d to 10.3.x. Thanks!
-
alexpott โ
committed e140d4a0 on 10.3.x
Issue #3436072 by omkar.podey, narendraR: Add validation constraints to...
-
alexpott โ
committed e140d4a0 on 10.3.x
-
alexpott โ
committed 9555002a on 11.x
Issue #3436072 by omkar.podey, narendraR: Add validation constraints to...
-
alexpott โ
committed 9555002a on 11.x
- ๐ฎ๐ณIndia omkar.podey
followup ๐ Either Remove default_summary_length config in text.settings or reuse it. Active
Automatically closed - issue fixed for 2 weeks with no activity.