- Issue created by @mparker17
- 🇨🇦Canada mparker17 UTC-4
For my own reference, in the merge request, the updated Setup for working on an issue is...
# Switch to the issue fork here ddev symlink-project ddev drush -y site:install minimal ddev drush -y pm:install toolbar field_ui views_ui ddev drush -y theme:install claro ddev drush -y config:set system.theme admin claro ddev drush -y pm:install elasticsearch_connector_test ddev drush -y elasticsearch_connector_test:add-test-content ddev drush -y search-api:index test_elasticsearch_index ddev drush -y cr ddev launch $(ddev drush uli)
And there is now a search view at...
/test-elasticsearch-index-search
- 🇨🇦Canada mparker17 UTC-4
These changes have been working quite well for me as I've been writing the following sets of documentation:
-
mparker17 →
committed a71553ea on 8.0.x
Issue #3496277 by mparker17: Make 8.0.x a bit easier to test
-
mparker17 →
committed a71553ea on 8.0.x
- 🇨🇦Canada mparker17 UTC-4
Realizing this will make testing 🐛 Merging Multiple Facet Conditions in facets_post_filters with OR Conjunction in Elasticsearch Connector Active easier, so I'm going to merge it now.
Automatically closed - issue fixed for 2 weeks with no activity.
- 🇨🇦Canada mparker17 UTC-4
Note this was released in elasticsearch_connector-8.0.0-alpha3 → .
- 🇨🇦Canada mparker17 UTC-4
The docs had already been updated: checking off that item in the issue summary.
- 🇨🇦Canada mparker17 UTC-4
I can unassign this from myself now that it is Closed (fixed)