-
Notifications
You must be signed in to change notification settings - Fork 17
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 support for Interfaces & Unions #5
Comments
@charliejrgower started working on this issue via WorksHub. |
Thinking in earnest -> Whereas something like subscriptions is basically all about execution side, and in practice these two are also mainly about putting some additional requirements on field resolvers and request structure, that''s a bit insignificant compared to what direction you are modelling from in the first place. But of course you know that. Issue has obviously been up for a long time, but was the bounty as well? :O subtle in plain sight!! In all seriousness though, and minding my whimsey and incorrect intuitions (see issue i opened) - what is the edge to trying to cover non-pred stuff more easily done in eg Lacinia straight, instead of stretching spec, which brings power but also limitations? Especially if several approaches can coexist :) |
Definitely too neurotic about types being "defined" by their specs, in the end we're just talking keywords. With the unions themselves autocreated as needed for a query of that kind instead of explicitly, seems neat enough. No children would be harmed because of an abstract mental dependency from resolver on own spec... haha. But the sorta passthrough any? specs likely for these queries, idunno. |
@Rastin2020 started working on this issue via WorksHub. |
A user started working on this issue via WorksHub. |
We would like to be able to use Interfaces and Unions in GraphQL. It's not immediately obvious how we might implement this, and we don't currently have a solid use case, but I understand it's a deal-breaker for some people who have considered Leona.
The text was updated successfully, but these errors were encountered: