-
Notifications
You must be signed in to change notification settings - Fork 189
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
[NodeSearchBunde] [SearchBundle] Add parameters for ES authentication and use them in version check #2120
Conversation
wesleylancel
commented
Sep 12, 2018
•
edited
Loading
edited
Q | A |
---|---|
Bug fix? | yes |
New feature? | no |
BC breaks? | no |
Deprecations? | no |
Fixed tickets | n/a |
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.
Hi @wesleylancel, your PR needs some changes
- Your answer if this PR is a bugfix seems to be incorrect.
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.
Hi @wesleylancel, your PR needs some changes
- This PR seems to need a milestone of a patch release.
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.
Hi @wesleylancel, your PR passed all our requirements.
Thank you for contributing!
* 5.0: update changelog Add parameters for ES authentication and use them in version check (#2120) [AdminBundle] use full version of CKeditor (#2119) KUN-3274 Show full path to file in form export (#2118) Fix bug in NodeChoiceType (#2115) [AdminBundle] fix icon sprite of ckeditor (#2099) Fixed issue when running kuma:user:create with --no-interaction flag (#2103) Fixed issue with locale in kuma:user:create command (#2104) [NodeSearchBundle] Fix search index of childpages of a structured node (#2091)
Please stop this madness. How much more fixes for this feature are you willing to make?
All of the above combines and enforces the first point: you should not do this kind of things in bundle extension. If you want to support numerous configuration formats, add two separate branches for it and use one or another, users choice! Or transform the old format into new format automatically! BTW, why ES version larger than 2 is automatically assumed 6? |