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

re #880 - Ensures that a nested filter consisting of only a filter does ... #881

Merged
merged 1 commit into from
Aug 19, 2014

Conversation

Philo
Copy link
Contributor

@Philo Philo commented Aug 14, 2014

...not result in a conditionless NEST filter (leading to its exclusion from generated json query)

…lter does not result in a conditionless NEST filter (leading to its exclusion from generated json query)
@gmarz
Copy link
Contributor

gmarz commented Aug 14, 2014

LGTM. Great catch @Philo ! Thanks for reporting and submitting a fix. Just need you to sign the CLA so we can pull it in, if you don't mind.

Also, for future pull requests, we recommend installing the EditorConfig Visual Studio extension for auto formatting to match our style guidelines.

@gmarz
Copy link
Contributor

gmarz commented Aug 18, 2014

Hey @Philo, just following up to see if you've had the chance to sign the CLA yet? Would really like to merge this in.

@Philo
Copy link
Contributor Author

Philo commented Aug 18, 2014

Yes, i've completed the CLA. Do i need to pass anything back to you from it?

@gmarz
Copy link
Contributor

gmarz commented Aug 18, 2014

Nope, nothing more to do on your part. There's sometimes a delay between signing the CLA and the confirmation on our end. Once I can confirm I'll pull this in. Thanks again.

@gmarz
Copy link
Contributor

gmarz commented Aug 19, 2014

@Philo, can you try signing the CLA one more time? Sorry, but for some reason the confirmation never came through.

@Philo
Copy link
Contributor Author

Philo commented Aug 19, 2014

@gmarz, ive completed the CLA, perhaps i didn't verify it correctly, hopefully will be fine now

@gmarz
Copy link
Contributor

gmarz commented Aug 19, 2014

@Philo, it just came in. Sorry for the delay!

gmarz added a commit that referenced this pull request Aug 19, 2014
re #880 - Ensures that a nested filter consisting of only a filter does ...
@gmarz gmarz merged commit e89f847 into elastic:develop Aug 19, 2014
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.

2 participants