Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add noThrow option to NetworkLayer advanced options #61

Merged
merged 4 commits into from Sep 6, 2018

Conversation

jamesone
Copy link
Contributor

@jamesone jamesone commented Sep 6, 2018

This ensures that no errors are thrown if noThrow = true. It brings the same noThrow functionality as: https://github.com/relay-tools/react-relay-network-modern#advanced-options-2nd-argument-after-middlewares

It's working for me 馃憤I purposely threw errors from my backend and it doesn't throw the error.

Useful links:

@nodkz nodkz merged commit 827ea30 into relay-tools:master Sep 6, 2018
@nodkz
Copy link
Collaborator

nodkz commented Sep 6, 2018

Great! Thanks a lot! 馃憤

@jamesone
Copy link
Contributor Author

jamesone commented Sep 6, 2018

@nodkz Don't you need to create an npm release (if you haven't already - maybe it's still processing)? https://www.npmjs.com/package/react-relay-network-layer (last published 3 months ago)

@jamesone
Copy link
Contributor Author

jamesone commented Sep 6, 2018

And my pleasure, it's a great library 馃憤

@nodkz
Copy link
Collaborator

nodkz commented Sep 6, 2018

Sorry, it should be done automatically via semantic-release. But by travis logs happened some error when publishing the package.

I get to work and fix it in an hour or two.

nodkz added a commit that referenced this pull request Sep 6, 2018
@nodkz
Copy link
Collaborator

nodkz commented Sep 6, 2018

Successfully published react-relay-network-layer@2.2.0

Sorry for the delay. Usually, CI publishes automatically a new version after merging. I repaired it, so for next PRs it should work as expected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants