Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Apollo Client documentation to use Apollo Client v3 imports #1282

Merged

Conversation

jerelmiller
Copy link
Contributor

Hello 👋

I am a maintainer on Apollo Client and happened to notice that the Apollo Client documentation is using the outdated v2 imports. I've updated the documentation to use the v3 imports in this PR in an effort to try and keep it consistent with the latest versions of Apollo Client. Hope this helps! Thanks for such a great library!

import * as AbsintheSocket from "@absinthe/socket";
import { createAbsintheSocketLink } from "@absinthe/socket-apollo-link";
import { Socket as PhoenixSocket } from "phoenix";
import { hasSubscription } from "@jumpn/utils-graphql";
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To avoid an additional dependency in the user's codebase, I've gone ahead and updated this example to use our recommendation in the directional composition section of our docs.

Copy link
Contributor

@benwilson512 benwilson512 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you so much for the contribution!

@benwilson512 benwilson512 merged commit 08be7ba into absinthe-graphql:master Nov 12, 2023
7 checks passed
@jerelmiller jerelmiller deleted the update-apollo-documentation branch November 14, 2023 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants