Skip to content
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

nestjs 10 support #113

Open
dalepo opened this issue Jun 26, 2023 · 5 comments
Open

nestjs 10 support #113

dalepo opened this issue Jun 26, 2023 · 5 comments

Comments

@dalepo
Copy link

dalepo commented Jun 26, 2023

No description provided.

@appinteractive
Copy link

appinteractive commented Jun 29, 2023

hey @dalepo any luck? It was working initially and now suddenly stopped working with v10.

@ntegral do you have some time looking into that?

I do get a bunch of GraphQLError`s with this module now, it complains about all schema entries:

GraphQLError: Input Object type UpdateUserInput must define one or more fields.
      at SchemaValidationContext.reportError (/Users/*/projects/nofapi/api/node_modules/graphql/type/validate.js:73:7)
      at validateInputFields (/Users/*/projects/nofapi/api/node_modules/graphql/type/validate.js:532:13)
      at validateTypes (/Users/*/projects/nofapi/api/node_modules/graphql/type/validate.js:260:7)
      at validateSchema (/Users/*/projects/nofapi/api/node_modules/graphql/type/validate.js:43:3)
      at graphqlImpl (/Users/*/projects/nofapi/api/node_modules/graphql/graphql.js:60:63)
      at /Users/*/projects/nofapi/api/node_modules/graphql/graphql.js:23:43
      at new Promise (<anonymous>)
      at graphql (/Users/*/projects/nofapi/api/node_modules/graphql/graphql.js:23:10)
      at GraphQLSchemaFactory.create (/Users/*/projects/nofapi/api/node_modules/@nestjs/graphql/dist/schema-builder/graphql-schema.factory.js:50:65)
      at GraphQLSchemaBuilder.generateSchema (/Users/*/projects/nofapi/api/node_modules/@nestjs/graphql/dist/graphql-schema.builder.js:35:52)
      at GraphQLSchemaBuilder.build (/Users/*/projects/nofapi/api/node_modules/@nestjs/graphql/dist/graphql-schema.builder.js:22:31)
      at GraphQLFactory.generateSchema (/Users/*/projects/nofapi/api/node_modules/@nestjs/graphql/dist/graphql.factory.js:27:69)
      at ApolloDriver.generateSchema (/Users/*/projects/nofapi/api/node_modules/@nestjs/graphql/dist/drivers/abstract-graphql.driver.js:23:36)
      at GraphQLModule.onModuleInit (/Users/*/projects/nofapi/api/node_modules/@nestjs/graphql/dist/graphql.module.js:109:54)
      at callModuleInitHook (/Users/*/projects/nofapi/api/node_modules/@nestjs/core/hooks/on-module-init.hook.js:51:9)
      at NestApplication.callInitHook (/Users/*/projects/nofapi/api/node_modules/@nestjs/core/nest-application-context.js:223:13) {
    path: undefined,
    locations: undefined,
    extensions: [Object: null prototype] {}

@liranasia
Copy link

also waiting for support, had to bump versions and now not working with nestjs/common version 10

@appinteractive
Copy link

hey @liranasia for me it was the missing cli plugin support when using swc with nest 10. I needed to add @field to every entity, arg, input properties and now it works.

@Kolahzary
Copy link

Currently @ntegral/nestjs-sentry is our only dependency which doesn't support nest 10:

image

@alumni
Copy link

alumni commented Mar 4, 2024

This should be fixed now.

However, I strongly recommend switching to @travelerdev/nestjs-sentry since it has a better architecture (graphql separation) and other other fixes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants