- Issue created by @Nikro
- ๐ช๐ธSpain Nikro Benalmadena, Malaga
Alright:
- Added the method to the interface
- Made sure base abstract method implements it with a NULL
- Adjusted the search backend itself to use the column if checkbox is checked
- ๐ฉ๐ชGermany marcus_johansson
There are some phpcs/cspell issues that needs to be addressed. The phpunit issue would be fixed by merging with 1.2.x
- ๐ฌ๐งUnited Kingdom scott_euser
I think this needs an update pulled from 1.2.x as the test coverage is failing. It should also be somewhat straightforward to add test coverage for this into AiSearchSetupMySqlTest
Thanks!
- ๐บ๐ธUnited States Kristen Pol Santa Cruz, CA, USA
We are doing some issue management housekeeping and adding/removing components.
We are removing the "Code" component and want people to categorize issues with the best module/submodule component.
Moving this issue to "AI Search".
See ๐ Update AI module project components Active for more details.
- Issue was unassigned.
- Status changed to Needs work
2 months ago 10:38am 30 July 2025 - First commit to issue fork.
- ๐ฎ๐ณIndia anjaliprasannan
AiSearchSetupMySqlTest is marked as old. in the codebase. should that be reverted? I have added test to AiSearchSetupMySqlTest but since this was named old I am reverting back. Once its confirmed, I will proceed.
- ๐ฌ๐งUnited Kingdom scott_euser
Hi! It's blocked at the moment due to an issue with Search API ๐ Reinstate MySQL tests Active
- ๐ฌ๐งUnited Kingdom scott_euser
Okay ๐ Reinstate MySQL tests Active is now fixed, rebased now
- ๐ฌ๐งUnited Kingdom scott_euser
Okay I believe what is needed left:
- An update to search_api_ai_search.backend.schema.yml to add the new config to the schema
- An update hook like ai_search_update_10003 which sets the new config default value to false
- ๐ฎ๐ณIndia abhisekmazumdar India
I can try to add the missing things here.
- ๐ฎ๐ณIndia abhisekmazumdar India
Made the required remaining changes listed by @scott_euser. Thanks for that! That made things easy for me to quickly jump in and add them.
Update the test. It looks like it is working now. I see a few failed cases for AiSearchSetupMySqlTest, but they are mostly annotation deprecation warnings and a few others that are not related to the changes included in this MR. - ๐ฉ๐ชGermany marcus_johansson
Thanks @abhisekmazumdar - I'll set to RTBC and if you have time Scott to check before Beta release, please do. The asked for fixes are there and I have tested it out.
For the failing tests we have ๐ AI Search tests is failing on next minor Active as a separate issue.
Now that this issue is closed, please review the contribution record.
As a contributor, attribute any organization helped you, or if you volunteered your own time.
Maintainers, please credit people who helped resolve this issue.
-
marcus_johansson โ
committed 2ee963d6 on 1.2.x authored by
nikro โ
Resolve #3525296 "Add optional raw"
-
marcus_johansson โ
committed 2ee963d6 on 1.2.x authored by
nikro โ