- First commit to issue fork.
- Status changed to Needs review
7 months ago 7:02pm 30 August 2024 - 🇫🇷France dqd London | N.Y.C | Paris | Hamburg | Berlin
Needs review with one question I still have in mind: Why we set 255 chars as default when we call field settings anyway? Wouldn't fetching field settings be enough? The default would be 255 neither then if not changed. I just copied the original patch into MR one to one for reference and credits to the original author. But possible optimizations can be discussed now.