-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Mutation reducers #404
Mutation reducers #404
Conversation
Slava
commented
Jul 15, 2016
•
edited
Loading
edited
- Update CHANGELOG.md with your change
- Make sure all of the significant new logic is covered by tests
- Rebase your changes on master so that they can be merged easily
- Make sure all tests and linter rules pass
29a50d3
to
77551cf
Compare
this pr no longer relies on #400 |
- Added a "noFetch" option to WatchQueryOptions that only returns available data from the local store (even it is incomplete). [Issue #225](https://github.com/apollostack/apollo-client/issues/225) and [PR #385](https://github.com/apollostack/apollo-client/pull/385). | ||
- Introduce a new (preferable) way to express how the mutation result should be incorporated into the store and update watched queries results: `updateQueries`. [PR #404](https://github.com/apollostack/apollo-client/pull/404). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Link the relevant issue?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd also add a new changelog entry that we made writing query results to store more efficient since they maintain object references in the case where data didn't change.
Looks great to me, just a few comments. Actually really hyped about how little code this was! |
Also needs to be rebased |
In response to Sashko's code review #404
In response to Sashko's code review #404
In response to Sashko's code review #404