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.

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

Support sealed classes for interfaces #645

Merged
merged 2 commits into from
Mar 21, 2020

Conversation

smyrick
Copy link
Contributor

@smyrick smyrick commented Mar 20, 2020

馃摑 Description

Allows sealed classes to be used as GraphQL Interfaces. This means that you can restrict the possible implementations of the interfaces since subclasses can only be declared with the root class

https://kotlinlang.org/docs/reference/sealed-classes.html

馃敆 Related Issues

Fixes #642

@smyrick smyrick added type: enhancement New feature or request changes: minor Changes require a minor version labels Mar 20, 2020
Copy link
Collaborator

@dariuszkuc dariuszkuc left a comment

Choose a reason for hiding this comment

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

@smyrick smyrick merged commit 20e512b into ExpediaGroup:master Mar 21, 2020
@smyrick smyrick deleted the sealed-class branch March 21, 2020 23:40
dariuszkuc pushed a commit to dariuszkuc/graphql-kotlin that referenced this pull request Aug 5, 2022
* Support sealed classes for interfaces

* Update docs and examples with more sealed classes

Co-authored-by: Shane Myrick <accounts@shanemyrick.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes: minor Changes require a minor version type: enhancement New feature or request
Development

Successfully merging this pull request may close these issues.

Sealed classes not treated as abstract
2 participants