Skip to content

Conversation

@marcus-nl
Copy link
Contributor

@marcus-nl marcus-nl commented Jul 30, 2021

Fixes #558

Checklist

  • Pull requests follows the contribution guide
  • New or modified functionality is covered by tests

Description

DictionaryTypeResolver now considers the full type hierarchy -- superinterfaces in addition to superclasses.

@marcus-nl marcus-nl changed the title Test case and should fix for #558 Test case and proposed fix for #558 Jul 31, 2021
Copy link
Collaborator

@oryan-block oryan-block left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix. I have one request.

@marcus-nl
Copy link
Contributor Author

Updated.
(I could not use animal -> pet -> dog, since Animal was already defined in the same package in TestInterfaces.kt)

@oryan-block oryan-block merged commit 89d7775 into graphql-java-kickstart:master Aug 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants