search

gramps-graphql / gramps--legacy

The core data source combination engine of GrAMPS.
https://gramps.js.org
MIT License
197 stars 13 forks source link

Let graphql-tools handle merging merge-resolvers #79

Closed mfix22 closed 6 years ago

mfix22 commented 6 years ago

As of graphql-tools@2.20.0, an array of merge resolvers can now be passed directly to mergeSchemas: https://github.com/apollographql/graphql-tools/compare/v2.19.0...v2.20.0

This PR updates GRAMPs to take advantage of this.

ecwyne commented 6 years ago

If this works, we should also remove lodash.merge as a dependency.

idibidiart commented 6 years ago

Could u guys test this with Union types? in other words it should work for __resolveType

mfix22 commented 6 years ago

No problem thanks for merging! Hope you are feeling more settled 😄