Skip to content
This repository was archived by the owner on Jan 26, 2021. It is now read-only.

Implement an equivalent for Parse.Query.count() #88

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

philikon
Copy link
Contributor

@philikon philikon commented Jul 6, 2015

For now it isn't backed by a store of any sorts, so apps would have to manually refresh the query. I think that's an acceptable situation for most apps. Going forward we should figure out if and how integration with client-side mutations makes sense.

@philikon
Copy link
Contributor Author

philikon commented Aug 3, 2015

Latest revision now stores the result of the count query on the subscription, alongside resultSet. This way, subsequent subscriptions to the same query will immediately receive the already stored result.

@philikon
Copy link
Contributor Author

Pingsies :)

@andrewimm andrewimm mentioned this pull request Sep 10, 2015
4 tasks
@TylerBrock
Copy link

All I want for Christmas is for this to get merged.

@philikon
Copy link
Contributor Author

🎁 🌲 🎁

@ghost
Copy link

ghost commented Jul 12, 2016

Thank you for your pull request. As you may know, we require contributors to sign our Contributor License Agreement, and we don't seem to have you on file and listed as active anymore. In order for us to review and merge your code, please email [email protected] with your details so we can update your status.

@ghost ghost added the CLA Signed label Aug 30, 2016
@ghost
Copy link

ghost commented Aug 30, 2016

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants