Skip to content

Fix multi search request #1658

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
Merged
merged 1 commit into from
May 6, 2022
Merged

Fix multi search request #1658

merged 1 commit into from
May 6, 2022

Conversation

miriam-eid
Copy link
Contributor

Fix: #1656

@miriam-eid miriam-eid requested a review from delvedor April 21, 2022 13:04
@github-actions
Copy link
Contributor

Following you can find the validation results for the API you have changed.

API Status Request Response
msearch 🟢 17/17 16/16

You can validate this API yourself by using the make validate target.

@sethmlarson sethmlarson merged commit f9dfbe8 into main May 6, 2022
@sethmlarson sethmlarson deleted the fix-1656 branch May 6, 2022 15:18
This was referenced May 6, 2022
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sethmlarson sethmlarson sethmlarson approved these changes

@delvedor delvedor Awaiting requested review from delvedor

Assignees
No one assigned
Labels
backport 7.17 backport 8.2 specification
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Sort missing from MultisearchBody
3 participants
@miriam-eid @sethmlarson @miriameid