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

Fixed Mutation error-state with errorPolicy:all #2297

Closed
wants to merge 4 commits into from

Conversation

LowieHuyghe
Copy link

With errorPolicy:all Mutation should call the correct callbacks and set the correct state.

Fix for #2253: errorPolicy: all in defaultOptions causes error to be undefined

@apollo-cla
Copy link

@LowieHuyghe: 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/

@amacleay
Copy link
Contributor

I would love to see this change get in - figuring out this issue has been a huge pain and I believe your change will fix it @LowieHuyghe !

Let me know if there's anything I can do to help!

@hwillson hwillson self-assigned this Sep 26, 2018
@hwillson
Copy link
Member

Thanks very much for working on this @LowieHuyghe! We're going to go with the slightly different approach outlined in #2374, but your contributions here helped form that PR. ❤️

@hwillson hwillson closed this Sep 28, 2018
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.

4 participants