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.

Sign up for GitHub

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

Jump to bottom

Added aggregations to response if requested #15

Merged
merged 7 commits into from
Mar 4, 2019
Merged

Added aggregations to response if requested #15

merged 7 commits into from
Mar 4, 2019

Conversation

snakeye
Copy link
Collaborator

@snakeye snakeye commented Aug 15, 2018

No description provided.

snakeye and others added 3 commits August 7, 2018 17:21
@snakeye
Added aggregations to response if requested
23b9dfe
@askoretskiy
Fixed doc_type check
3bcbc4a 
doc_type to `search` method is list, while `document['_type']` is a string.
@snakeye
Merge pull request #1 from askoretskiy/patch-1
bcc113a 
Fixed doc_type check
@vrcmarcos
Copy link
Owner

Hi,
Thank you for your contribution!

Could you please rebase it and fix the build to pass the Travis checks? :)

@coveralls
Copy link

Coverage Status

Coverage decreased (-3.2%) to 95.484% when pulling 3cd13b2 on snakeye:master into 7ad1551 on vrcmarcos:master.

@coveralls
Copy link

coveralls commented Oct 7, 2018 •
edited
Loading

Coverage Status

Coverage increased (+0.04%) to 98.83% when pulling 6861516 on snakeye:master into 3a31753 on vrcmarcos:master.

@snakeye
Copy link
Collaborator Author

snakeye commented Oct 7, 2018

Done.

@vrcmarcos vrcmarcos merged commit f068f52 into vrcmarcos:master Mar 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@snakeye @vrcmarcos @coveralls @askoretskiy