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

[CR-22] Update search_api_solr #126

Open
wants to merge 1 commit into
base: 8.x-2.x
Choose a base branch
from
Open

[CR-22] Update search_api_solr #126

wants to merge 1 commit into from

Conversation

AndreyMaximov
Copy link

@AndreyMaximov AndreyMaximov commented Sep 10, 2020

This is the same as #99 but with the updated Solr configuration set.

Original Issue, this PR is going to fix: https://fivejars.atlassian.net/browse/CR-22

Steps for review

General checks

  • verify composer checkouts drupal/search_api_solr 4.1.x and solarium/solarium 6.x

Installing Open Y from scratch

  • During installation enable Open Y Search API with Solr backend
  • At the Configure Solr Search step, verify the configuration form contains correct default values:
    • Solr path - empty
    • Solr core - collection1
  • Verify the installation process finishes well

Enabling Solr search on the Open Y website

  • go to any build where Open Y is installed, but the Open Y Solr Search is not installed yet
  • login as admin
  • enable Open Y Search API (and Search API Solr Legacy for fj builds)

Updating Open Y with Solr search enabled

  • Verify the update goes smoothly
  • Verify there are instructions posted during the upgrade (for Drush)
  • Verify there are messages that the Solr configuration is outdated (for FJ builds, see [CR-22] Open Y Search API Solr upgrade #99)
  • Verify there are no messages that the Solr configuration is outdated (for FJ builds where the configuration is updated, see [CR-22] Update search_api_solr #126 - this PR )

@AndreyMaximov AndreyMaximov added NEEDS REVIEW Code review is needed NEEDS TESTING The functional testing is needed labels Sep 10, 2020
@AndreyMaximov AndreyMaximov added the DO NOT MERGE Do not merge please label Sep 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DO NOT MERGE Do not merge please NEEDS REVIEW Code review is needed NEEDS TESTING The functional testing is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants