docs: new path for useServer v6 and up#8087
docs: new path for useServer v6 and up#8087jchanke wants to merge 1 commit intoapollographql:mainfrom
Conversation
`useServer` should be imported from the path `graphql-ws/use/ws` instead of `graphql-ws/lib/use/ws`, starting from graphql versions 6.0.0 and up. see enisdenjo/graphql-ws#617
|
@jchanke: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Apollo Contributor License Agreement here: https://contribute.apollographql.com/ |
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. |
|
Hi, are you sure that's the only change? In addition to another import in the same file, the code shows a few calls to I can try to make the other changes myself but as I don't really actively use this WS support, it's easier to validate when the PR comes from somebody successfully using graphql-ws. Are you able to audit the calls to (We really only have this doc page as a replacement for the fact that we had subscriptions built in way back in Apollo Server v2; at some point we should just delete it and simply have a sentence somewhere noting that |
|
Thanks for pointing that out. I'll look into that as well! |
useServershould be imported from the pathgraphql-ws/use/wsinstead ofgraphql-ws/lib/use/ws, starting from graphql versions 6.0.0 and up.see enisdenjo/graphql-ws#617