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
temporarily make delegationContext optional until next major version #1614
Conversation
The latest changes of this PR are available as alpha in npm: Quickly update your package.json by running:
|
Seems to fix https://github.com/oxilor/graphql-tools-issue-1605 |
But addition of specifiedBy directive is causing some -- but not all?? -- graphql v15 versions to fail testing |
@yaacovCR Fixed it! |
And the plan would be to move the Transform types to delegate in the next major version and make the argument types explicit, correct? Anything better we can do? |
@yaacovCR Whatever you think is better option, I am okay with that :) |
I think we should push this for now as fixes current bugs, but there might be a better workaround later prior to next major release. |
includes also changes to transform method arguments see #1614
includes also changes to transform method arguments see #1614
includes also changes to transform method arguments see #1614
includes also changes to transform method arguments see #1614
includes also changes to transform method arguments see #1614
includes also changes to transform method arguments see #1614
includes also changes to transform method arguments see #1614
includes also changes to transform method arguments see #1614
includes also changes to transform method arguments see #1614
includes also changes to transform method arguments see #1614
includes also changes to transform method arguments see #1614
includes also changes to transform method arguments see #1614
includes also changes to transform method arguments see #1614
includes also changes to transform method arguments see #1614
includes also changes to transform method arguments see #1614
includes also changes to transform method arguments see #1614
includes also changes to transform method arguments see #1614
includes also changes to transform method arguments see #1614
Addresses typing issues in #1605