- Issue created by @anjaliprasannan
- 🇩🇪Germany marcus_johansson
@AnjaliPrasannan - added a minor comment to fix, otherwise it looks good.
- First commit to issue fork.
- Issue was unassigned.
- Status changed to RTBC
2 days ago 9:38am 14 May 2025 - 🇮🇳India divyansh.gupta Jaipur
I was able to reproduce this issue and this MR solved the issue for me,
Thus moving this to RTBC!! - 🇩🇪Germany marcus_johansson
That code is very good and we can merge it, since you can uninstall providers as well.
Another the big issue is that we (or I :D) forgot to set the assistant as required on the chatbot block I think?
You shouldn't be able to create an AI Assistant entity without a default provider, thus it shouldn't be possible to setup the block without and assistant.
Could we just add that required as well to the form?
- 🇮🇳India divyansh.gupta Jaipur
Made the Ai assitant field required:
Please review. - 🇮🇳India Ishani Patel
Hello @marcus_johansson and @divyansh.gupta
The issue is already in RTBC for making the AI assistant field require:
Please check this Issue link : https://www.drupal.org/project/ai/issues/3520273 🐛 Make an AI Assistant select list field required on AI Chatbot block configuration page ActiveThank you!
-
marcus_johansson →
committed 7819600c on 1.1.x authored by
anjaliprasannan →
#3512736 Fix for null check.
-
marcus_johansson →
committed 7819600c on 1.1.x authored by
anjaliprasannan →