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

If all field of an interface are private an invalid schema is generated #870

Closed
Andy2003 opened this issue Jan 25, 2022 · 2 comments · Fixed by #875
Closed

If all field of an interface are private an invalid schema is generated #870

Andy2003 opened this issue Jan 25, 2022 · 2 comments · Fixed by #875
Labels
bug Something isn't working confirmed Confirmed bug high priority

Comments

@Andy2003
Copy link
Contributor

Given the following schema (taken from the tests):

interface UserInterface {
  private: String @private
}

type User implements UserInterface {
  id: ID
  password: String @private
  private: String
}

An empty interface is generated:

interface UserInterface

Which is not allowed by the GrapQL spec:

An Interface type must define one or more fields.

@Andy2003 Andy2003 added the bug Something isn't working label Jan 25, 2022
@neo4j-team-graphql
Copy link
Collaborator

Many thanks for raising this bug report @Andy2003. 🐛 We will now attempt to reproduce the bug based on the steps you have provided.

Please ensure that you've provided the necessary information for a minimal reproduction, including but not limited to:

  • Type definitions
  • Resolvers
  • Query and/or Mutation (or multiple) needed to reproduce

If you have a support agreement with Neo4j, please link this GitHub issue to a new or existing Zendesk ticket.

Thanks again! 🙏

@neo4j-team-graphql neo4j-team-graphql added the confirmed Confirmed bug label Jan 25, 2022
@neo4j-team-graphql
Copy link
Collaborator

We've been able to confirm this bug using the steps to reproduce that you provided - many thanks @Andy2003! 🙏 We will now prioritise the bug and address it appropriately.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working confirmed Confirmed bug high priority
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants