-
Notifications
You must be signed in to change notification settings - Fork 186
-
Notifications
You must be signed in to change notification settings - Fork 186
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
schema stitching #230
Comments
Sorry, I've missed, that merge-grahpql-schemas is deprecated. I'll take a look. No EDD yet. |
👍 Let us know if we can help |
@bboure I'll definitely do it, feel free to assign the issue to me. I'm just not sure when exactly. Approximately this weekend. |
@bboure How far are you with integrating Amplify CLI https://github.com/aws-amplify/amplify-cli/tree/master/packages/amplify-category-api? It looks promising and if you're already working on integrating it, we might avoid one more intermediate step and concentrate our efforts on Amplify CLI. I'd be glad to help here as well. |
@trilliput There is no integration to Amplify CLI on the roadmap yet There are definitely things that we could explore and that were discussed already, like GraphQL Transformation, for generating resources, data sources and mapping templates. There is also Codegen Maybe we could use it somehow under the hood to auto generate Optionaly, we could add something on top of AWS-cli that does the equivalent of All ideas and PRS are welcome :) |
Does this mean we cannot use the |
I believe we basically need this to convert Apollo schemas into AppSync version. Dug very deep on Sunday and made I don't think I can finish it on my own, looking for contributors to cover all directives and edge cases. |
Closing this issue as |
We recently merged #227 which adds a great addition to the plugin.
However, I just realized that it relies on a deprecated package: https://github.com/okgrow/merge-graphql-schemas
I suggest, we refactor the PR and use Appolo Stitching instead.
@trilliput would you have time to have a look at this?
The text was updated successfully, but these errors were encountered: