- Issue created by @brunodbo
- ๐ฎ๐ณIndia divyansh.gupta Jaipur
I tested this and confirmed that the Automator Text Suggestion plugin does not appear for string_long / string_textarea fields until theyโre added to the plugin annotation.
After adding them, the automator works as expected.Before preparing an MR: should we extend support to all text-related field + widget types (e.g. text_long, text_with_summary, text_textarea_with_summary) at the same time, so coverage is complete? Or keep the MR focused only on string_long / string_textarea?
- ๐จ๐ฆCanada brunodbo
Since it seems this is a general purpose text automator, I think it should work for all text fields/widgets. I would say let's include all of them, since otherwise the same issue will arise for other text fields/widgets as well.
- Merge request !841Issue #3541931: Add support for various field types. โ (Merged) created by divyansh.gupta
- ๐ฎ๐ณIndia divyansh.gupta Jaipur
Iโve updated the plugin annotation to cover all core text field types and widgets (string, string_long, text, text_long, text_with_summary and their widgets). Tested and confirmed the Automator now shows up for each of them.
Please review!! - ๐จ๐ฆCanada brunodbo
The attribute code in the MR looks good as far as I can tell. I also manually tested the
automator_text
plugin on all text fields/widgets, and the automator was available for configuration in the form display settings, and worked on the node form for all of them.Sidenote: I did run into an issue with the
text_with_summary
field type. In the form display, it's possible to configure two field widget actions for such a field, but in the field's settings, you can only configure a single automator, so both field widget actions need to use the same prompt etc., which isn't desirable in all cases (especially since the summary automator needs a prompt that tells the LLM to summarize). Additionally, when I only configured thesummary_textarea_with_summary
automator on the field to try it out, it wasn't working for me on the node form (no text was entered in the summary field). These are probably separate issues though, we should get one of the maintainers to confirm. Leaving this as "Needs review" for now. - ๐ฉ๐ชGermany marcus_johansson
Thank you, looking good and getting merged!
-
marcus_johansson โ
committed 302ba82a on 1.2.x authored by
divyansh.gupta โ
Issue #3541931: Add support for various field types.
-
marcus_johansson โ
committed 302ba82a on 1.2.x authored by
divyansh.gupta โ