keystone
keystone copied to clipboard
keystonejs
make context config in ApolloServer config works
🦋 Changeset detected
Latest commit: ea612116fa96c1322747df21d60e33caaea6290b
The changes in this PR will be included in the next version bump.
This PR includes changesets to release 1 package
| Name | Type |
|---|---|
| @keystone-ui/core | Patch |
Not sure what this means? Click here to learn what changesets are.
Click here if you're a maintainer who wants to add another changeset to this PR
![changeset-bot[bot] avatar](https://avatars.githubusercontent.com/in/28616?v=4 "Published by a pub.dev verified publisher"){kind=small}
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/keystonejs/keystone-next-docs/9DQtu88rjc2VD9H9qSSbG8JmUPXA
✅ Preview: https://keystone-next-docs-git-fork-bichikim-update-a-63a6b4-keystonejs.vercel.app
![vercel[bot] avatar](https://avatars.githubusercontent.com/in/8329?v=4 "Published by a pub.dev verified publisher"){kind=small}
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.
![codesandbox-ci[bot] avatar](https://avatars.githubusercontent.com/in/38409?v=4 "Published by a pub.dev verified publisher"){kind=small}
Hi @bichikim,
Thanks for the pull request, I think this highlights a problem we need to fix, but maybe not exactly like this pull request. I think I see two options
- Restrict the Typescript types to not include an Apollo
context, and update our documentation - Alternatively, add or merge the Apollo
contextinto our own Keystonecontext
