- Issue created by @bisonbleu
- Merge request !727Create 'Email' FieldWidgetAction for AiAutomatorType LlmEmail → (Merged) created by Unnamed author
- 🇨🇦Canada bisonbleu
Strangely, in my local ddev env. both phpcs and phpstan run without finding issues, but in GitLab phpcs fails !? I'm running these commands.
% ddev exec vendor/bin/phpcs --standard=Drupal,DrupalPractice web/modules/contrib/ai/modules/ai_automators/src/Plugin/FieldWidgetAction/Email.php
% ddev exec vendor/bin/phpstan analyze web/modules/contrib/ai/modules/ai_automators/src/Plugin/FieldWidgetAction/Email.php Note: Using configuration file /var/www/html/phpstan.neon. 1/1 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100% [OK] No errors
- 🇨🇦Canada bisonbleu
With the latest commit, multiple emails are extracted from a Basic Page Body field. The only issue is I can't quite get/figure the ajax to update the multi-item field. In the video, I simply refreshed the page to see all 3 items.
The other issue, is that another call to the LLM occurs on Save, and that should not happen. See 🐛 When Field Widget worker is selected, Direct also fires (?) Active