Skip to content

Any chance for a input union support? #18

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

Closed
TheServerAsterisk opened this issue Jan 17, 2017 · 4 comments
Closed

Any chance for a input union support? #18

TheServerAsterisk opened this issue Jan 17, 2017 · 4 comments

Comments

@TheServerAsterisk
Copy link
Contributor

TheServerAsterisk commented Jan 17, 2017

Even though it is not officially supported by the current specs, there is currently an unofficial implementation for it. Personally, I think it would be great if it was there was perhaps a feature or is there a good workaround for this issue?

@mhallin
Copy link
Member

mhallin commented Jan 19, 2017

Interesting. We could maybe support it with a feature flag. Are there any other GraphQL implementations that support it?

@TheServerAsterisk
Copy link
Contributor Author

TheServerAsterisk commented Jan 20, 2017

No, as far as I can tell there doesn't seem to be direct support for it in other implementation or even indirect support, but many of these are projects are quite young like this one.

There might of have been some indirect support for it in the Go implementation but overlooked it due to a lack of familiarity with the language (not likely though). While I have figured out a different way to approach the data, it just would have been nicer and less complex. I'll leave it up to you on what should be done.

@mhallin
Copy link
Member

mhallin commented Apr 19, 2017

After thinking on this and working some more on an application, it seems as if a limited version of input unions that "automatically" maps to tagged Rust enums would be interesting. It would be a subset of the specification linked above.

I will try to work on this for a bit. The tricky part is to not build something that a later GraphQL will make obsolete :)

@theduke
Copy link
Member

theduke commented May 16, 2019

Le'ts not support anything not in the spec, we have enough catch-up work to do as is.

We can re-open if/when this gets accepted.

@theduke theduke closed this as completed May 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants