Skip to content

Gracefully handle checkVersions network error#201

Merged
hwillson merged 4 commits intoapollographql:masterfrom
mjlyons:mjlyons/handle-checkversion-network-error
May 31, 2019
Merged

Gracefully handle checkVersions network error#201
hwillson merged 4 commits intoapollographql:masterfrom
mjlyons:mjlyons/handle-checkversion-network-error

Conversation

@mjlyons
Copy link
Copy Markdown
Contributor

@mjlyons mjlyons commented May 8, 2019

Apollo Dev Tools is failing to initialize for me because my site's CSP settings block the checkVersions() request (to https://devtools.apollodata.com/graphql).

This change allows init() to proceed if the checkVersions network request fails.

I've verified that Apollo Dev Tools starts working with the patch by building the extension locally.

@apollo-cla
Copy link
Copy Markdown

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

Copy link
Copy Markdown
Member

@hwillson hwillson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great idea - thanks @mjlyons!

@hwillson hwillson merged commit 06a6f17 into apollographql:master May 31, 2019
woahitsraj pushed a commit to woahitsraj/apollo-client-devtools that referenced this pull request Oct 15, 2019
* Gracefully handle checkVersions network error

* Slight wording tweak

* Changelog update
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.

3 participants