-
-
Notifications
You must be signed in to change notification settings - Fork 1k
feat: add @vue/apollo
example implementation
#70
Conversation
Co-authored-by: Renovate Bot <bot@renovateapp.com>
@danielroe Do you plan on finishing example with server or should we do it later? (asking to avoid keeping stalled PR) |
@pi0 Thanks for the ping. I think |
Sure. Please ping me when ready to squash (since WIP is locked for merge) |
@pi0 Should be good to go now (though note issue with transpilation). |
Current issue with native esm:
It may just be an issue anyone will face with esm - implementing workaround but it will need to be fixed upstream I think. |
1aeb417
to
ee8eaef
Compare
Closing as is stalled. Will rework later on (probably to nitro demo or fresh example) |
@pi0 @danielroe I would be interested in providing an up-to-date GraphQL example. What is the problem with this PR? I was also wondering about the best way to create the graphql server with h3. This example here uses the express middleware which seems to be overkill. Maybe its worth to create a special h3 middleware and add it to https://www.apollographql.com/docs/apollo-server/integrations/middleware/? |
@tobiasdiez Indeed. We even made apollo express middleware work for Apollo but it is overkill (https://github.com/nuxt/nitro-demo/blob/main/server/graphql.js) so PR welcome there if you want to improve it and enable it back in demo. Finally, I want to move integration to [nuxt-community/apollo-module)(https://github.com/nuxt-community/apollo-module) with bridge and nitro support this way others need to have just a module |
This is now updated to work with new examples layout, and with a sample graphql server backend to show it in action.
We currently need to transpile
@vue/apollo-composables
as it does not provide a CJS build but only AMD or ESM (see vuejs/apollo#1081).closes nuxt/nuxt#11729