Skip to content
This repository was archived by the owner on Jan 28, 2022. It is now read-only.

Remove graphql-tag dependency from docs #58

Merged
merged 3 commits into from
Jun 10, 2021
Merged

Conversation

akd-io
Copy link
Contributor

@akd-io akd-io commented Jun 10, 2021

Apollo Client v3 comes with an export of graphql-tag.

The @apollo/client package includes graphql-tag as a dependency and re-exports gql. To simplify your dependencies, we recommend importing gql from @apollo/client and removing all graphql-tag dependencies.

https://www.apollographql.com/docs/react/migrating/apollo-client-3-migration/#graphql-tag

I've refactored all occurrences in the docs to use the gql tag of @apollo/client except in the recommended list on the @nhost/nuxt page, where I'm unsure if it's still useful, as I'm not well-versed in Vue-land. I quote:

Other recommended NPM packages: nhost-js-sdk @nuxtjs/apollo graphql-tag

All occurrences can be found here:
https://github.com/nhost/docs/search?q=graphql-tag

Hope this is useful, and thank you so much for this wonderful platform!

@vercel
Copy link

vercel bot commented Jun 10, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/nhost/docs/GDbTpRZacUyucQtwtmN1K8AvuyvT
✅ Preview: https://docs-git-fork-akd-io-master-nhost.vercel.app

@elitan
Copy link
Contributor

elitan commented Jun 10, 2021

Perfect!

Related to what we need to do with our @nhost/react-apollo package: nhost/react-apollo#7

@elitan elitan merged commit 389e613 into nhost:master Jun 10, 2021
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.

2 participants