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

docs(graphql): Update id directive for interfaces (#209) #382

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

MichelDiz
Copy link
Contributor

Fixes DOC-275

* Update content/graphql/schema/ids.md

Co-authored-by: Abhimanyu Singh Gaur <12651351+abhimanyusinghgaur@users.noreply.github.com>
@CLAassistant
Copy link

CLAassistant commented Dec 15, 2022

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ rderbier
❌ bucanero
You have signed the CLA already but the status is still pending? Let us recheck it.

@netlify
Copy link

netlify bot commented Dec 15, 2022

Deploy Preview for dgraph-docs-repo ready!

Name Link
🔨 Latest commit 22b0df7
🔍 Latest deploy log https://app.netlify.com/sites/dgraph-docs-repo/deploys/640f50247b25680008d39ce1
😎 Deploy Preview https://deploy-preview-382--dgraph-docs-repo.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@MichelDiz MichelDiz marked this pull request as ready for review April 25, 2023 00:44
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.
Copy link
Contributor

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 {

Copy link
Contributor

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?

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

Successfully merging this pull request may close these issues.

None yet

5 participants