- Issue created by @wouters_f
- Merge request !381[3497673] add tag so in these prompts tokens can be overridden bu external modules → (Merged) created by wouters_f
- 🇬🇧United Kingdom MrDaleSmith
The changes in the MR don't seem to correspond to the description of the ticket, and I'm not sure what they actually add: could you provide a little more explanation of what will be different after this issue is merged, and why?
- 🇬🇧United Kingdom MrDaleSmith
This looks like it might be an either/or/why not both? situation with https://www.drupal.org/project/ai/issues/3485686 ✨ PreGenerateResponseEvent should allow altering of tags Active which is also waiting for review.
- 🇩🇪Germany marcus_johansson
I think both issues makes sense, but as for this issue, if we only want the module name, I think a better solution is that we add it in the providerproxy - we already add the operation type there, see: https://git.drupalcode.org/project/ai/-/blob/1.0.x/src/Plugin/ProviderPr...
So adding a method to the baseclass that gets the module name and setting this in the ProviderProxy. That way module name would always be given without any effort for 3rd party modules.
- 🇩🇪Germany marcus_johansson
So since the previous thing I wrote is not true, we will go ahead and merge this.
Will look into [# https://www.drupal.org/project/ai/issues/3485686] → as well.