-
Notifications
You must be signed in to change notification settings - Fork 150
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
Top level interface connection #4821
Conversation
🦋 Changeset detectedLatest commit: 7fcd0aa The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
@@ -53,44 +53,11 @@ export class InterfaceEntityAdapter { | |||
this.initConcreteEntities(entity.concreteEntities); | |||
} | |||
|
|||
public findAttribute(name: string): AttributeAdapter | undefined { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This bit here is just reordering some methods in InterfaceEntityAdapter to make it more consistent
Thanks for the Neo4j GraphQL Toolbox updates. The Neo4j GraphQL Toolbox has now been torn down - reopening this PR will republish it. |
0262c7d
to
862a34d
Compare
862a34d
to
579e751
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, maybe some tck tests could be useful?
Performance ReportNo Performance Changes Show Full Table
Old Schema Generation: 26.154s |
Description
Support for top-level connection query on interfaces. For example:
Typedefs
Query