- Issue created by @Martygraphie
- ๐ฉ๐ชGermany marcus_johansson
Thanks @martygraphie - I would suggest the following solution:
- There is an attribute called OperationType - we add an optional description to it, of TranslatableMarkup or string type.
- When showing this in the AI Settings module, we show the attrbutes description method.
- For the operation type + capability like Chat with Tools, we just added the information specifically in the form.
- ๐ฎ๐ณIndia divyansh.gupta Jaipur
Made the changes suggested,
Please review!! - ๐ฉ๐ชGermany marcus_johansson
Thanks @divyansh.gupta - I added some comments, we need these strings to be translatable everywhere, otherwise it looks good!
- ๐ฎ๐ณIndia divyansh.gupta Jaipur
@marcus_johansson, Made the changes suggested on the MR,
Please review!! - ๐ฎ๐ณIndia divyansh.gupta Jaipur
Converted the hard coded selections into a function, such that it does not cause problem when we edit the order in future.
Please review!! - ๐ฎ๐ณIndia annmarysruthy
@divyansh.gupta Kindly resolve conflict in file src/Form/AiSettingsForm.php and also this MR includes commits unrelated to the issue. Could you please rebase your branch.
- ๐ฎ๐ณIndia divyansh.gupta Jaipur
Actually the MR was pointing wrong target branch, also rebased the branch,
Please review!! - ๐ฎ๐ณIndia divya.sejekan
Verified using MR!716 , The help texts are added as per mentioned in description.
In Drupal AI module , Discussion is named as Chat . Here in description it is mentioned as Discussion.Keeping for further Code review since there is merge error
- Status changed to Needs review
8 days ago 1:31pm 21 October 2025