-
Notifications
You must be signed in to change notification settings - Fork 59
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
feature: @oneOf input object support #537
Conversation
Added support for oneOf from graphql-js and new template renderer
✅ Docs Preview ReadyNo new or changed pages found. |
✅ Deploy Preview for apollo-ios-docc canceled.
|
✅ Deploy Preview for eclectic-pie-88a2ba ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor comment, but otherwise this looks good!
apollo-ios-codegen/Sources/ApolloCodegenLib/Templates/OneOfInputObjectTemplate.swift
Outdated
Show resolved
Hide resolved
Updating the SPMInXcodeProject codegen config to commit the animal kingdom package.swift and use local version of library
This reverts commit bb3cce4.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good Zach, thank you!
@@ -57,7 +57,7 @@ | |||
"path" : "./AnimalKingdomAPI", | |||
"moduleType" : { | |||
"swiftPackageManager" : { | |||
|
|||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this one be changed to the new swiftPackage
too?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yea might as well, not actually required since this gets decoded into the new one but worth going ahead and switching it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually looks like this one may require the change because of the need for the local path causing the ci failure
Looks like there is still a CI test failure - is that CI related or more work to be done? |
git-subtree-dir: apollo-ios git-subtree-mainline: 4c87fc4 git-subtree-split: daa2c1e408304c4943d12f5f1445b4fb2e7f4254
…pport git-subtree-dir: apollo-ios-codegen git-subtree-mainline: 8c28971 git-subtree-split: 461d092e5835dcfadf43aaa37606305582d1ac59
Adding support for @OneOf Input Objects, more info can be found in the official RFC here: graphql/graphql-spec#825
Closes apollographql/apollo-ios#3430