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

Expose updateQuery #152

Closed
wants to merge 5 commits into from
Closed

Expose updateQuery #152

wants to merge 5 commits into from

Conversation

joarwilk
Copy link

@joarwilk joarwilk commented Aug 18, 2016

@apollo-cla
Copy link

@joarwilk: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Meteor Contributor Agreement here: https://contribute.meteor.com/

@jbaxleyiii
Copy link
Contributor

@joarwilk thanks for this! Could you add a usage test as well?

@joarwilk
Copy link
Author

joarwilk commented Aug 18, 2016

@jbaxleyiii Done. Do note it breaks. The updateQuery method was supposed to be a part of apollo client 0.4.11 but it didn't make it somehow. This can't be merged until the next client release.

I'm not sure how you guys handle breaking changes but I guess this would have to wait until the next minor (?) react-apollo version goes out since it depends on a newer client version.

@jbaxleyiii jbaxleyiii self-assigned this Aug 30, 2016
@jbaxleyiii
Copy link
Contributor

@joarwilk could you rebase this? I'd love to get it merged and out for release!

@jbaxleyiii jbaxleyiii mentioned this pull request Sep 6, 2016
@jbaxleyiii jbaxleyiii closed this Sep 6, 2016
@jbaxleyiii
Copy link
Contributor

I pulled this into #152 so I could go ahead and release it!

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.

3 participants