Skip to content
This repository was archived by the owner on Apr 13, 2023. It is now read-only.

Conversation

jbaxleyiii
Copy link
Contributor

@jbaxleyiii jbaxleyiii commented May 6, 2016

This PR brings support for polling with the client as well as passes stopPolling and startPolling to the client.

It also adds a check to prevent rerenders if the data doesn't change.

@jbaxleyiii jbaxleyiii changed the title [WIP] update apollo-client Update apollo-client and test polling May 6, 2016
Feature: add `startPolling` and `stopPolling` to the prop object for queries
Bug: Fix bug where full options were not being passed to watchQuery

### v0.3.1
Copy link
Contributor

Choose a reason for hiding this comment

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

Why two separate versions?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

0.3.1 was already published but not updated in the changelog

@jbaxleyiii jbaxleyiii changed the title Update apollo-client and test polling [WIP] Update apollo-client and test polling May 6, 2016
@jbaxleyiii jbaxleyiii changed the title [WIP] Update apollo-client and test polling Update apollo-client and test polling May 6, 2016
@jbaxleyiii
Copy link
Contributor Author

jbaxleyiii commented May 6, 2016

@stubailo this should be good to go. If you are good with it, feel free to merge. It'll deploy on its own

@jbaxleyiii jbaxleyiii changed the title Update apollo-client and test polling Update apollo-client, increase performance, and pass polling props May 6, 2016
@@ -1,9 +1,9 @@
declare module "lodash.isobject" {
import main = require('~lodash/index');
export = main.isObject;
// export = main.isObject;
Copy link
Contributor

Choose a reason for hiding this comment

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

What is this about?

@stubailo
Copy link
Contributor

stubailo commented May 6, 2016

Looks good to me! Only thing is I'm not sure what's with the commented typings.

@jbaxleyiii
Copy link
Contributor Author

Oh yeah! I upgraded my typings locally. I think it was seeing ambient.d.ts and the generated files. I'm not really sure, need to investigate more

@stubailo
Copy link
Contributor

stubailo commented May 6, 2016

:shipit: looks good!

@jbaxleyiii jbaxleyiii merged commit 01e0e27 into master May 6, 2016
@jbaxleyiii jbaxleyiii deleted the update-client branch May 6, 2016 02:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants