search

silverstripe / cwp-core

CWP basic compatibility module
BSD 3-Clause "New" or "Revised" License
3 stars 12 forks source link

Separate out FulltextSearch components to their own repository #14

Closed NightJar closed 6 years ago

NightJar commented 6 years ago

Fulltext functionality is intended to be optional to the site's requirements, however is currently an imposed requirement. While it is not necessary to be used, it must be installed in order for a dev/build (or any other flushing activity) to succeed.

This issue is of two parts, one to separate out what is contained in this cwp-core repository, and the other to affect the cwp repository: https://github.com/silverstripe/cwp/issues/21

Ideally there'll be e.g. cwp/cwp-search or suchlike that would house the adaptive functionality that enables the features provided by/for the platform, rather than essentially having an unlisted dependency for this module.

robbieaverill commented 6 years ago

@tractorcow how do you feel about us moving the fulltextsearch compatibility code from both cwp/cwp-core and cwp/cwp into a new module, e.g. cwp/cwp-search?

tractorcow commented 6 years ago

Yes that would be fine. Could you also shift all the basepage code into an extension in that case?

robbieaverill commented 6 years ago

Pull requests added to the description of https://github.com/silverstripe/cwp/issues/21