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

Use a valid filter() for userId Datastore query #70

Open
wants to merge 1 commit into
base: master
from

Conversation

@dbs
Copy link

@dbs dbs commented Mar 22, 2016

The code for the solution for the Datastore query that is supposed to return
only a specific user's books fails to return any books. The problem is that
the comparison operator and property key were included in a single string.
Break the values out into separate strings so that the filter() method has
a property, comparison operator, and value as expected.

Closes #69.

Signed-off-by: Dan Scott denials@gmail.com

The code for the solution for the Datastore query that is supposed to return
only a specific user's books fails to return any books. The problem is that
the comparison operator and property key were included in a single string.
Break the values out into separate strings so that the filter() method has
a property, comparison operator, and value as expected.

Closes #69.

Signed-off-by: Dan Scott <denials@gmail.com>
@googlebot
Copy link

@googlebot googlebot commented Mar 22, 2016

We found a Contributor License Agreement for you (the sender of this pull request) and all commit authors, but as best as we can tell these commits were authored by someone else. If that's the case, please add them to this pull request and have them confirm that they're okay with these commits being contributed to Google. If we're mistaken and you did author these commits, just reply here to confirm.

@dbs dbs changed the title Use a 3-arg filter() for a Datastore query Use a valid filter() for userId Datastore query Mar 22, 2016
@dbs
Copy link
Author

@dbs dbs commented Mar 22, 2016

I'm Dan Scott, and I confirm that I did author this commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.