- Issue created by @Vivek Panicker
- Merge request !328Issue #3493474: Add back missing Vector DB explorer link. → (Merged) created by Vivek Panicker
- 🇩🇪Germany marcus_johansson
Reviewed and its passing the testing. Thanks Vivek!
- 🇫🇷France mattlc
Hello. Thank @vivek-panicker you for this issue.
Maybe this would be simpler to use foreach here as $key and array_key_exists() does not seems to bring any value ?
if ($indexes = $this->entityTypeManager->getStorage('search_api_index')->loadMultiple()) { /** @var \Drupal\search_api\IndexInterface $index */ foreach ($indexes as $index) { $backend = $index->hasValidServer() ? $index->getServerInstance()->getBackendId() : NULL; if ($backend === 'search_api_ai_search') { $return = TRUE; break; } } }
- 🇮🇳India Vivek Panicker Kolkata
Hey @mattlc!
I agree that's a better solution to the current one.
Would you like to implement it? - 🇫🇷France mattlc
MR updated.
Don't know why CSpell now fails in pipeline complaining with word 'outro' in src/Form/AiSetupForm.php. - 🇮🇳India Vivek Panicker Kolkata
I believe we can ignore that one since it's not related to the work we are doing here.
- 🇮🇳India Vivek Panicker Kolkata
Tested the patch.
"Vector DB Explorer" link appears for me on the `/admin/config/ai/explorers` page.
No errors in dblog also.
The patch is good to go. - 🇩🇪Germany marcus_johansson
Thank you both, I merged with the latest dev branch to fix the testing issues and will merge this now.
-
marcus_johansson →
committed 70d4b68e on 1.0.x authored by
vivek panicker →
Issue #3493474: Add back missing Vector DB explorer link.
-
marcus_johansson →
committed 70d4b68e on 1.0.x authored by
vivek panicker →
-
marcus_johansson →
committed 70d4b68e on 3489690-add-an-event authored by
vivek panicker →
Issue #3493474: Add back missing Vector DB explorer link.
-
marcus_johansson →
committed 70d4b68e on 3489690-add-an-event authored by
vivek panicker →
Automatically closed - issue fixed for 2 weeks with no activity.