-
Notifications
You must be signed in to change notification settings - Fork 16
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
browser-ponyfill not fixed #56
Comments
I habe the same problem here. I think graphql-request is still importing cross-fetch, even if this module has dropped the dependency since 7.0.1. I also tried to override fetch via config options, however I think also this is broken at the moment, as no options get passed to the graphql-request client: export default defineNuxtConfig({
modules: ['nuxt-graphql-request'],
graphql: {
useFetchPolyfill: true,
clients: {
default: {
endpoint: 'XXX',
options: {
responseMiddleware: responseMiddleware,
fetch: fetch,
},
}
}
},
runtimeConfig: {
public: {
graphql: {
clients: {
default: {
endpoint: 'XXX',
options: {
responseMiddleware: responseMiddleware,
fetch: fetch,
},
},
},
},
},
},
}); When I dump {
clients: {
default: { endpoint: 'XXX', options: {} }
}
} |
I think it is not fixed in 7.0.1 and these tickets are related ... #55 |
@Gomah can I assist you in fixing the issue? |
Unfortunately, that's inheriting from the build: {
transpile: ['nuxt-graphql-request'],
}, |
What does transpile do? |
It transpiles the declared dependencies using Babel instead.
|
Still getting this error in 7.0.4 with Nuxt 3.8.1. I was able to find a workaround by adding this to nuxt.config.ts:
I do not have anything in my my build transpile array. |
@jovrtn same thing with nuxt 3.8.4, your fix working for me |
#54 (comment)
The requested module '/_nuxt/node_modules/.pnpm/[email protected]/node_modules/cross-fetch/dist/browser-ponyfill.js?v=7c73fe01' does not provide an export named 'default
The text was updated successfully, but these errors were encountered: