Skip to content

Conversation

brunoocasali
Copy link
Member

@brunoocasali brunoocasali commented Jun 5, 2023

Include a number of fixes after the latest breaking changes.

faceted_search_2: |-
await client.multiSearch(MultiSearchQuery(queries: [
SearchQuery(
IndexSearchQuery(
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's missing the query parameter

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep, I've opened an issue about it #321

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've added the query params, but I think we should fix the issue in any case.

@brunoocasali brunoocasali force-pushed the fix-code-samples branch 3 times, most recently from 27199c0 to dbea512 Compare June 5, 2023 16:33
@brunoocasali brunoocasali added the documentation Improvements or additions to documentation label Jun 5, 2023
alallema
alallema previously approved these changes Jun 6, 2023
Copy link
Contributor

@alallema alallema left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🌮

@brunoocasali
Copy link
Member Author

bors merge

@brunoocasali
Copy link
Member Author

bors cancel

@meili-bors
Copy link
Contributor

meili-bors bot commented Jun 6, 2023

Canceled.

meili-bors bot added a commit that referenced this pull request Jun 6, 2023
322: Fix code-samples after #310 release r=brunoocasali a=brunoocasali

Include a number of fixes after the latest breaking changes.

Co-authored-by: Bruno Casali <brunoocasali@gmail.com>
@brunoocasali brunoocasali merged commit 3592870 into main Jun 6, 2023
@brunoocasali brunoocasali deleted the fix-code-samples branch June 6, 2023 19:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants