-
Notifications
You must be signed in to change notification settings - Fork 66
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
docs(graphql): Update id directive for interfaces (#209) #382
base: main
Are you sure you want to change the base?
Conversation
* Update content/graphql/schema/ids.md Co-authored-by: Abhimanyu Singh Gaur <12651351+abhimanyusinghgaur@users.noreply.github.com>
|
✅ Deploy Preview for dgraph-docs-repo ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
In this case, the `@id` field in the interface won't be unique for the interface but for each of its implementing types. | ||
This allows two different types implementing the same interface to have the same value for the inherited `@id` field. | ||
|
||
There are scenarios where this behavior might not be desired, and you may want to constrain the `@id` field to be unique across all the implementing types. In that case, you can set the `interface` argument of the `@id` directive to `true`, and Dgraph will ensure that the field has unique values across all the implementing types of an interface. |
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.
limit the width of sentence
@@ -63,8 +63,33 @@ query { | |||
|
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.
@rderbier told me that this feature does not work on v23. Do I understand it correctly?
Fixes DOC-275