-
Notifications
You must be signed in to change notification settings - Fork 58
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
Add gqlgen implementation #42
Conversation
@@ -9,6 +9,7 @@ The following open-source GraphQL server libraries provide support for Apollo Fe | |||
| Language | Framework | _service | @key (single) | @key (multi) | @key (composite) | @requires | @provides | ftv1 | | |||
| --- | --- | --- | --- | --- | --- | --- | --- | --- | | |||
| Elixir | [absinthe_federation](https://github.com/DivvyPayHQ/absinthe_federation) | ✔️ | ✔️ | ✔️ | ✔️ | ✔️ | ✔️ | ❌ ([coming soon](https://github.com/DivvyPayHQ/absinthe_federation/pull/25)) | | |||
| Go | [gqlgen](https://gqlgen.com/) | ✔️ | ✔️* | ✔️* | ❌ | ✔️ | ✔️ | ❌ | |
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.
Currently, gqlgen does not support multiple @key directives per entity, but I may try to put a PR together for it
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.
Yeah that is what the *
is for, it means that it supports @key(fields: "id")
(single) or @key(fields: "sku package")
(multi) but not @key(fields: "id") @key(fields: "sku package")
together at the same time
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.
It also looks like support for @key (composite)
just merged in the last 24hrs 🎉 Once that has been tagged we can update this to reflect that support
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.
FYI, @key (multi)
has been merged as well (not part of a release yet).
@kdawgwilk Man, you're a life saver. This is exactly what I needed right now. |
Closes #17