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

Upgrade dependencies to latest versions and fixed breaking changes. #9

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

Conversation

D1no
Copy link

@D1no D1no commented Apr 23, 2019

Node 10 is supported for a while now by grpc. Firebase 5.x incorporates that jump; nothing changed in regards to APIs for apollo-link-firebase but without this fix, it will always pull in a second version of firebase which will fail on Node 10.

Node 10 is supported for a while now by grpc. Firebase 5.x incorporates that jump; nothing changed in regards to APIs for `apollo-link-firebase` but without this fix, it will always pull in a second version of firebase which will fail on Node 10.
D1no added a commit to D1no/fightback that referenced this pull request Apr 23, 2019
Directive typing could be improved in the future by using apollo-utilities types instead of the constructed DirectiveArgs. That would require a refactor.
@D1no D1no changed the title Update Firebase Dependency to next Major for Node 10 compatibility. Upgrade dependencies to latest versions and fixed breaking changes. Apr 24, 2019
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.

None yet

1 participant