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

Fix batch fetch result on error #636

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

felipenr
Copy link

Maybe fix some issues for who uses "apollo-link-batch-http"

Problem:

Apollo client does not returns "data" values when has error on fetch. Even if I set errorPolicy 'all' or 'ignore'.

Cause:

"apollo-link-batch-http" returns Array result on fetch and not a Object.

Solution:

Fix condition to returns data on fetch error.

@apollo-cla
Copy link

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

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