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

Use client store. #222

Merged
merged 1 commit into from
Sep 22, 2016
Merged

Use client store. #222

merged 1 commit into from
Sep 22, 2016

Conversation

therealcisse
Copy link
Contributor

Just in case one's using an immutable store.

Just in case one's using an immutable store.
@apollo-cla
Copy link

@amsayk: 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

@amsayk could you add these changes to the changelog?

@therealcisse
Copy link
Contributor Author

@jbaxleyiii I updated the change log in another pull request #225. Also opened issue #224 to track this. Feel free to close when you merge this.

@therealcisse
Copy link
Contributor Author

@jbaxleyiii

On another subject, it would be great to have react-apollo work out of the box with immutable like redux-form by exposing react-apollo/immutable.

I don't know how feasible it would be or if its best for performance. But if you're open to it, I could send PR.

@jbaxleyiii
Copy link
Contributor

@amsayk what are your thoughts on how to accomplish it? I'm open to it for sure!

@jbaxleyiii jbaxleyiii merged commit 44aef41 into apollographql:master Sep 22, 2016
jbaxleyiii pushed a commit that referenced this pull request Sep 22, 2016
* Update Changelog.md

* Update Changelog.md
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