- Issue created by @Lendude
- Status changed to Needs review
over 1 year ago 1:26pm 1 March 2023 - 🇳🇱Netherlands Lendude Amsterdam
Sorry that was supposed to say maxlength_js_label, updated.
So something like this should work.
- 🇧🇪Belgium joevagyok
Thanks @Lendude! How did you discover this issue, have you encountered a problem?
The proposed solution looks good for me, but I think we should do the same formaxlength_js_label_summary
too. - 🇳🇱Netherlands Lendude Amsterdam
I ran into this while adding translations, since the original text was a multiline text, the translation needed to be that too, but currently that isn't possible for the label.
The summary was empty in our case, so missed that was an issue too, but I think you are right, that should be type 'text' too since that also uses a text area for entry in
maxlength_field_widget_third_party_settings_form
- Status changed to Needs work
over 1 year ago 4:48pm 24 March 2023 - 🇺🇸United States cedewey Denver, CO
Thanks for catching this and writing a patch Lendude. Marking this as Needs Work since the patch needs to be updated to include maxlength_js_label_summary as well.
- Status changed to Needs review
over 1 year ago 11:08am 25 March 2023 - 🇺🇸United States cedewey Denver, CO
Updated the patch to define maxlength_js_summary as text as well.
- Assigned to joevagyok
- Status changed to RTBC
over 1 year ago 4:16pm 14 April 2023 - @joevagyok opened merge request.
-
joevagyok →
committed 8c25576d on 2.1.x
Issue #3345225 by joevagyok, Lendude, cedewey: Config schema type for...
-
joevagyok →
committed 8c25576d on 2.1.x
- Status changed to Fixed
over 1 year ago 4:19pm 28 April 2023 Automatically closed - issue fixed for 2 weeks with no activity.