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!: remove hint from QueryRecordsResponse #368

Merged
merged 2 commits into from
Jul 28, 2022

Conversation

yuryu
Copy link
Member

@yuryu yuryu commented Jul 24, 2022

What type of PR is this?

/kind fix

What this PR does / Why we need it:
BREAKING CHANGE: Deletes hint from QueryRecordsResponse.
QueryRecordsResponse has a hint field but it should not have been there...
Hints are something passed from the client not returned by the server.
It's been around since #123 but never been implemented.

Which issue(s) this PR fixes:

Closes #367

Special notes for your reviewer:

Deletes hint from QueryRecordsResponse.
QueryRecordsResponse has a hint field but it should not have been there...
Hints are something passed from the client not returned by the server.
It's been around since googleforgames#123 but never been implemented.
@yuryu yuryu requested review from hongalex and a user July 24, 2022 00:41
@yuryu yuryu self-assigned this Jul 24, 2022
@yuryu yuryu merged commit 1fa6909 into googleforgames:main Jul 28, 2022
@yuryu yuryu deleted the fix/query-hint branch July 28, 2022 00:00
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.

QueryRecordsResponse should not include hint field
2 participants