- Issue created by @useernamee
- Merge request !112#3493780: Support for Search API view as configured through drupal_cms_search reciple. → (Open) created by useernamee
- 🇸🇮Slovenia useernamee Ljubljana
Ugh, I came full circle and ended with implementing changes from 📌 Use CE Generator to build View rows Needs work .
- 🇦🇹Austria fago Vienna
so this is about supporting search api views - let's make this clearer.
- 🇦🇹Austria fago Vienna
Quick review:
* I don't think the module README is the place for drupal-CMS documentation. The README should stay general and document how to use it with Search API. But concrete drupal-CMS instructions should go elsewhere imo (not sure where), or better be not necessary since automated, but that'S a different story/issue.
* Cannot judge about the details, but let's make sure things work via automated tests. Can we also add some search-api & views config to tests that we run as part of automated tests? I think we could simply copy things over from the recipe, but I'd copy to document what is supported on the long-time and not depend on the recipe since the recipe can change over time.