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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Throw exception on multi-level interface #420

Merged
merged 2 commits into from
Oct 16, 2019
Merged

Throw exception on multi-level interface #420

merged 2 commits into from
Oct 16, 2019

Commits on Oct 16, 2019

  1. Throw exception on multi-level interface 

    Resolves ExpediaGroup#419

According to the spec, an interface can not implement another interface. This is not caught by our schema generator but instead just returns the first level interface as the type in the schema. We are now throwing an exception to help translate this understanding of GraphQL through the Kotlin code
    @smyrick
    smyrick authored and Shane Myrick committed Oct 16, 2019
    Configuration menu
    Copy the full SHA
    09deb24 View commit details
    Browse the repository at this point in the history

  2. Return superclasses from multiple levels deep

    Shane Myrick committed Oct 16, 2019
    Configuration menu
    Copy the full SHA
    449b198 View commit details
    Browse the repository at this point in the history