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(recommend): Add Type RecommendQueriesResponse from old MultipleQueriesResponse for recommend #1472

Merged
merged 3 commits into from
Jul 20, 2023

Conversation

raed667
Copy link
Contributor

@raed667 raed667 commented Jul 18, 2023

In recommend add type MultipleQueriesResponse that basically reverts back to the old definition before PR #1460

@codesandbox-ci
Copy link

codesandbox-ci bot commented Jul 18, 2023

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit bb251cc:

Sandbox Source
javascript-client-app Configuration

@raed667 raed667 changed the title fix(recommend): Type MultipleQueriesResponse for recommend fix(recommend): Add Type RecommendQueriesResponse from old MultipleQueriesResponse for recommend Jul 19, 2023
@raed667 raed667 requested a review from Haroenv July 19, 2023 12:06
Haroenv
Haroenv previously approved these changes Jul 19, 2023
Copy link
Contributor

@Haroenv Haroenv left a comment

Choose a reason for hiding this comment

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

however, this should have a test to avoid the issue you make the PR for

@raed667
Copy link
Contributor Author

raed667 commented Jul 19, 2023

@Haroenv added a test that fails when the old type is used but passes when the new one is set.

Copy link
Member

@shortcuts shortcuts left a comment

Choose a reason for hiding this comment

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

gg!

@shortcuts shortcuts merged commit 36285b2 into master Jul 20, 2023
@shortcuts shortcuts deleted the fix/recommend-type-multiple-queries-response branch July 20, 2023 09:02
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