search

mickhansen / graphql-sequelize

GraphQL & Relay for MySQL & Postgres via Sequelize
MIT License
1.9k stars 172 forks source link

Remove relay peer dependency #655

Closed mschipperheyn closed 4 years ago

mschipperheyn commented 5 years ago

This library has a peer dependency on relay, which doesn't make a lot of sense if you use Apollo. Is it possible to remove it?

mickhansen commented 5 years ago

It has a peer dependency on graphql-relay as we use methods from that helper library: https://github.com/mickhansen/graphql-sequelize/blob/master/src/relay.js#L1

mschipperheyn commented 5 years ago

Ok, yeah, makes sense I guess. Are there any limitations to be considered in relation to Apollo v2?

mickhansen commented 5 years ago

Apollo is just the client/server interface right? relay in this instance refers primarily to the relay graphql specification (nodes and connections), there's nothing specific to how your client is setup.

stale[bot] commented 4 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.