Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added Solr to Lando with a script to fetch the latest Solr config #93

Merged
merged 3 commits into from
Oct 19, 2022

Conversation

stephen-cox
Copy link
Member

@stephen-cox stephen-cox commented Oct 19, 2022

This PR adds Solr to Lando for the localgov_search_solr module.

To test it you will need to composer require localgovdrupal/localgov_search_solr and enable it.

Fixes #86

@stephen-cox stephen-cox requested review from ekes and finnlewis October 19, 2022 09:58
@ekes
Copy link
Member

ekes commented Oct 19, 2022

To test it you will need to composer require localgovdrupal/localgov_search_solr and enable it.

Do we want to include localgovdrupal/localgov_search_solr in the project then? So not it localgovdrupal/localgov but here?

Copy link
Member

@ekes ekes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Manual test (and maybe useful for c&p for documentation):-

## first lines probably only for testing fresh like now
$ lando rebuild # needed to get the solr container
$ lando drush si -y localgov # needed to reinstall the site
$ lando composer require localgovdrupal/localgov_search_solr # have suggested maybe we want to do this in the project
$ lando drush en localgov_search_solr # enable solr search for the site
$ lando update-solr-config # update the config
$ lando restart # reload the config

Done!

Only other note. It's adding the configuration to the directory. So anything that's already there like if the default config with English and Irish it stays there even when just English is enabled in the new export.

Copy link
Member

@finnlewis finnlewis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This works for me locally and I love it!

Thanks @stephen-cox and @ekes and @andybroomfield !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Solr config in Lando setup
3 participants