-
Notifications
You must be signed in to change notification settings - Fork 115
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
TypeError: loader.loaderId is not a function #1807
Comments
Same here. Looks like something to do with invoking Add a console log to the issue area. LoadersRegistry.prototype.register = function (loader) {
console.log(loader, loader.loaderId);
if (!this._loaders.some(function (l) { return l.loaderId() === loader.loaderId(); })) {
this._loaders.push(loader);
}
};
// Output
GraphQLFileLoader {} [Function: loaderId]
UrlLoader {} [Function: loaderId]
JsonFileLoader {} [Function: loaderId]
GraphQLFileLoader {} [Function: loaderId]
CodeFileLoader { config: {} } undefined |
Here is the issue: ardatan/graphql-tools@9c26b84#r54088878 |
Related: ardatan/graphql-tools#3246 Looks like the current solution (for |
The current workaround is to use |
What is the last known good version? |
I took a look at this: ardatan/graphql-tools#3246 (comment) We have this PR to update it to the latest version (4.1.0), but until it gets merged I just had to lock "resolutions": {
"graphql-config": "4.1.0"
} |
any news on this? |
@rburgst This repo hasn't been updated since Aug 20, 2021, except deps updates. I don't know if they are maintaining this or not. |
@rburgst yes, It deprecated: https://the-guild.dev/graphql/codegen/docs/migration/graphql-cli |
Describe the bug
Running
graphql codegen
, get the error:Versions (please complete the following information):
The text was updated successfully, but these errors were encountered: