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

Fix Elasticsearch NGINX stub #464

Merged
merged 1 commit into from
Feb 19, 2020
Merged

Fix Elasticsearch NGINX stub #464

merged 1 commit into from
Feb 19, 2020

Conversation

samgranger
Copy link
Contributor


I have read the contribution guidelines and am targeting the branch <YOUR_TARGET_BRANCH>:
Because this is a Bug Fix which is Backwards Compatible.
Because this is a Feature which is not Backwards Compatible.
Because this is a Deprecation which is Backwards Compatible.
etc...

Changed

  • Fix logic to copy Elasticsearch NGINX stub on install

@mischabraam mischabraam merged commit a3a036e into weprovide:master Feb 19, 2020
@clnt
Copy link

clnt commented Mar 28, 2020

@mischabraam Hi, why hasn't this been tagged in a new minor release? It broke my fresh install this morning when pulling in version 2.0.0. I literally have a wiki of valet-plus issues compiled and the fixes for them, this one was about to get added.

Could we get more consistent releases for fixes like this please? all this needed was a 2.0.1 and I wouldn't have encountered the issue this morning.

Thanks.

@Neodork
Copy link
Collaborator

Neodork commented Apr 23, 2020

@clnt If you follow the contribution guidelines I would love to merge all your PR's. And as we get more PR's we can more often release smaller updates.

mischabraam pushed a commit that referenced this pull request Nov 25, 2020
* Update PULL_REQUEST_TEMPLATE.md with correct format

Updated the format to reflect the correct questions to be asked for a PR.

* Update readme.md

* Update readme.md

* Update readme.md

* Update readme.md

* Fix Elasticsearch NGINX stub (#464)

* Update readme.md

* Moved contributors to their own section.

* [TASK] Applied nginx changes based on Magento 2 Example NGINX

https://github.com/magento/magento2/blame/2.4-develop/nginx.conf.sample#L184

Co-authored-by: Lou van der Laarse <[email protected]>
Co-authored-by: Frank Houtappels <[email protected]>
Co-authored-by: Sam Granger <[email protected]>
Co-authored-by: Mischa Braam <[email protected]>
Co-authored-by: Sander <[email protected]>
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.

5 participants