- Issue created by @rajeshreeputra
- @rajeshreeputra opened merge request.
- ๐ฆ๐นAustria drunken monkey Vienna, Austria
Thanks for suggesting these changes!
However, I donโt think either of these modules, Search API Solr โ or SearchStax Search API Module โ , are actual dependencies of this module.While the module does offer some functionality that is specific to Solr, the tracking/analytics support is completely independent from that and will work for any Search API searches.
The documentation you link to is specifically for setting up a new search using a SearchStax server. (Maybe the documentation should reflect that better, though.) Also, that documentation already clearly states that the SearchStax Search API Module โ is optional, depending on how your search server is configured.Furthermore, from what I can tell it is not necessary to create a
composer.json
file if it merely mirrors the information in theMODULE.info.yml
file, as the Drupal Composer repository will already fill in all of that information.
What we could do, though, is add these two modules assuggest
ed packages incomposer.json
. - ๐ฎ๐ณIndia rajeshreeputra Pune
Thank you @drunken monkey, will move these modules from require to suggested.
-
drunken monkey โ
committed b6b46a93 on 1.x authored by
rajeshreeputra โ
Issue #3481005 by rajeshreeputra, drunken monkey: Added Search API...
-
drunken monkey โ
committed b6b46a93 on 1.x authored by
rajeshreeputra โ
- ๐ฆ๐นAustria drunken monkey Vienna, Austria
Looks good, thanks!
Merged. Automatically closed - issue fixed for 2 weeks with no activity.