-
Notifications
You must be signed in to change notification settings - Fork 34
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
IBX-1699: Bump requirement for Solr #228
Conversation
Co-authored-by: Andrew Longosz <alongosz@users.noreply.github.com>
Review fixes: Readme, and removed sample configs. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Requires @adamwojs' review.
This reverts commit f14f883.
Kudos, SonarCloud Quality Gate passed! |
Reverted f14f883 as it breaks the build, probably a regexp actually, so it won't work like that. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
QA approved on eXPlatform 2.5 (with ezplatform-solr-search-engine 2.0).
Install Solr 8.11.1 by default, as this includes the upgraded log4j with an important security fix.
For 2.0 branch, eZ Platform v2.5 (as of ezsystems/ezplatform#677 )
Work in progress:
Readme update is incompleteGood enough. Little need up to date manual install intructions when we anyway refer to the script that does this for us anyway.