Account created on 29 August 2019, almost 6 years ago
  • Engineer - Backend Drupal at QED42 
#

Merge Requests

More

Recent comments

🇮🇳India prabha1997

Hi @sirclickalot,
I think this is the same issue that was already reported here 🐛 Automators Tool entity type needs to be installed Active . But the problem is still happening. Can you please check it again? We can reopen the issue if needed.

🇮🇳India prabha1997

I’m also encountering the same issue.
Steps to Reproduce:
Install the following modules:

  1. ai, ai_provider_openai, ai_agents, ai_assistant_api, ai_chatbot
  2. Create an assistant and select any agent from the list.
  3. Set the agent to be used as the assistant.
  4. Place the AI Chatbot block in the block layout.
  5. Try interacting with the chatbot

Observed Issue:
When sending a message to the chatbot, the following error is thrown:

TypeError: Drupal\ai\OperationType\Chat\ChatMessage::__construct(): 
Argument #2 ($text) must be of type string, null given, 
called in /var/www/html/modules/contrib/ai/modules/ai_assistant_api/src/Service/AgentRunner.php on line 42
🇮🇳India prabha1997

I have not tested the functionality, but I have resolved all PHPCS issues.

🇮🇳India prabha1997

prabha1997 made their first commit to this issue’s fork.

🇮🇳India prabha1997

I’m currently unable to test the image_and_audio_to_video explorer, as I couldn’t find any provider listed for it in the documentation.

🇮🇳India prabha1997

@anjaliprasannan – I’m unable to reproduce this error after applying the changes from #3526029.

🇮🇳India prabha1997

Reviewed the MR and tested the changes. The MR fixes the issue when adding ai assistant without ai_agent installed

🇮🇳India prabha1997

@Prashant
There are two fields:
File field – Only .mp3 files are allowed.
Image field – Accepts .jpg, .jpeg, and .png formats.

🇮🇳India prabha1997

prabha1997 changed the visibility of the branch ai_agent-3521706-cannot-load-file to hidden.

🇮🇳India prabha1997

That issue was created to address a missing button.

🇮🇳India prabha1997

prabha1997 changed the visibility of the branch 3525028-speech-to-speech to hidden.

🇮🇳India prabha1997

prabha1997 changed the visibility of the branch 3525025-missing-button-on to hidden.

🇮🇳India prabha1997

Hi @marcus_johansson,
Thanks for sharing access — I’ve checked the sheet. While I understand the data, I’m not yet at the level where I feel confident analyzing and comparing the cost-effectiveness between 4.1 and 4o in depth.

🇮🇳India prabha1997

prabha1997 changed the visibility of the branch 3521769--ai-settings to hidden.

🇮🇳India prabha1997

As of now, translation is working correctly with only the required change applied. I’ve removed the extra check and unrelated modifications, keeping the fix minimal and focused on the actual issue.

🇮🇳India prabha1997

It worked for me after making the change suggested by @valthebald.
I've attached screenshots for reference.
before

after

🇮🇳India prabha1997

The issue was resolved after configuring the provider for 'Translate Text' in the AI default settings.

🇮🇳India prabha1997

Hi marcus_johansson,

We've removed the AiTranslateForm.php file, and I'm able to translate content without any issues.
I've attached a reference video for your review.

🇮🇳India prabha1997

Updated the $time property type in ContentTranslationControllerOverride to match the parent class (ContentTranslationController). The parent class defines it as TimeInterface, so it cannot be overridden as ?TimeInterface due to strict type compatibility in PHP. This change resolves the type mismatch error and maintains proper inheritance.

🇮🇳India prabha1997

prabha1997 changed the visibility of the branch 3482645-copy-reference-translated-entity to active.

🇮🇳India prabha1997

prabha1997 changed the visibility of the branch 3482645-copy-entity-reference to hidden.

🇮🇳India prabha1997

prabha1997 changed the visibility of the branch 3482645-copy-reference-translated-entity to hidden.

🇮🇳India prabha1997

prabha1997 changed the visibility of the branch 3482645-copy-reference-translated-entity to active.

🇮🇳India prabha1997

prabha1997 changed the visibility of the branch 3482645-copy-reference-translated-entity to hidden.

🇮🇳India prabha1997

prabha1997 changed the visibility of the branch 3482645-ai-translate-module to hidden.

🇮🇳India prabha1997

prabha1997 made their first commit to this issue’s fork.

🇮🇳India prabha1997

Hi @marcus_johansson,
I'm currently unable to access the spreadsheet — it looks like I don't have permission. Could you please grant me access?
Thanks!

🇮🇳India prabha1997

Hi @marcus_johansson,
The previous merge request was created against the 1.0.x branch. I’ve now created a new branch based on the 1.1.x branch.
Kindly review it. Thanks!

🇮🇳India prabha1997

prabha1997 changed the visibility of the branch 3482645-copy-entity-reference to active.

🇮🇳India prabha1997

prabha1997 changed the visibility of the branch 3482645-copy-entity-reference to hidden.

🇮🇳India prabha1997

Yes, You can close this issue as "Won't do", since #3521499 addresses the concern.

🇮🇳India prabha1997

Thank you for your feedback.
I've now added the following assertion to cover the invalid email case with a space:
$this->assertFalse($validator->isValid('example @example.com'));
Please let me know if any further changes are needed.

🇮🇳India prabha1997

Thank you for the feedback! As of now, I’m getting the default models from the plugin and have set them dynamically like this. Please review this approach. I understand your point about editorial choices, and regarding bumping models to GPT-4.1, we can wait until the tests with Drupal CMS are completed. Once we have more insights, we can make the necessary changes.

🇮🇳India prabha1997

Hello @ishani,

Thank you for checking the 1.1.x branch.

You're right — I reviewed it on my end, and the "Use source vocabulary" field does appear when a content type includes a taxonomy reference field. In my earlier test, I had created a node type without any taxonomy reference field, which is why the option was not visible.

I also verified this with the Article content type, which uses its own tag vocabulary — and as expected, the field was available.

Thanks again for the clarification!

🇮🇳India prabha1997

I have tested this issue. Please review the before and after screenshots below for reference.

🇮🇳India prabha1997

I have tested this issue. Please review the before and after videos below for reference.

🇮🇳India prabha1997

Thanks @AnjaliPrasannan it is working now as expected

🇮🇳India prabha1997

I’ve created a new issue to track this and I’m tagging it here for reference.
https://www.drupal.org/project/ai/issues/3523519 🐛 The "Use source vocabulary" field is currently missing from the AI Content Suggestions section when editing content. Active

🇮🇳India prabha1997

This issue has been fixed in the 1.0.x branch.
However, in the 1.1.x branch, the "Use source vocabulary" field is missing.

🇮🇳India prabha1997

I’ve added a dynamic implementation to set the default provider and model ID for each operation type.

🇮🇳India prabha1997

I’ve tested all explorers after the changes, and everything is working as expected.

🇮🇳India prabha1997

I tried it on 1.1.x of ai branch, and it is working as expected. Please refer to the attached screenshot for reference.

🇮🇳India prabha1997

prabha1997 made their first commit to this issue’s fork.

🇮🇳India prabha1997

prabha1997 made their first commit to this issue’s fork.

🇮🇳India prabha1997

prabha1997 made their first commit to this issue’s fork.

Production build 0.71.5 2024