Upgrade graphql and typescript dependencies #1
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR was born to upgrade graphql-ws and apollo-client dependencies and became a fight with the dependency hell of javascript. What we did:
graphql-ws
andapollo-client
Some dependencies had breaking changes so we managed to fix them in the most easy way.
'@apollo/client'
were refactored in'@apollo/client/core
, since the core part does not include React, and we were having problems importing react (that actually we don't need).src/subscription.ts
now usesshouldRetry
instead ofisFatalConnectionProblem
as a configuration to enable retries on fatal errors, since the latter is deprecatedV
in the methodssubscribe
,createQuery
,createMutation
since the underlying library added some constraints on it