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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Support for Interface types in SDL-Transformation and generade ad… #12

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mwildt
Copy link

@mwildt mwildt commented Mar 19, 2021

Trying around with this package, i found that it is not possible to inject federation information for interface-types. I made some minor changes in the transform-sdl file to support ferderation of interface types.

I also added generation of @extends directives.

Looking forward to your feedback...

Malte Wildt added 2 commits March 19, 2021 22:22
@@ -89,4 +89,6 @@ typings/
# DynamoDB Local files
.dynamodb/

dist/
Copy link

Choose a reason for hiding this comment

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

@mwildt Is it required to remove git ignore for folder dist? You also added a lot of new files in oder dist/* which shouldn't belong to this pull request.

@johgoe
Copy link

johgoe commented Oct 26, 2021

In general I tested to extend a GraphQLInterface type for federation succesful.

I hope the changes in 78567ae in the files

  • src/transform-sdl.ts
  • src/transform-federation.ts

will be merged to this package to make it available in general.

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

Successfully merging this pull request may close these issues.

2 participants