Skip to content

add DEPTH symbol#1764

Closed
yaacovCR wants to merge 2 commits intomasterfrom
fix-errors
Closed

add DEPTH symbol#1764
yaacovCR wants to merge 2 commits intomasterfrom
fix-errors

Conversation

@yaacovCR
Copy link
Collaborator

in preparation for fixing #1047

yaacovCR added 2 commits July 13, 2020 02:01
specify functions rather than strings, with the gateway fieldNode as a parameter, returning the required selection set for the target schema, allows passing arguments from the gateway to the target schema.
@yaacovCR
Copy link
Collaborator Author

Work in progress, backwards compatible version of #1650 to fix #1047

@yaacovCR
Copy link
Collaborator Author

;) abandoning for now. Fix will require a lot of ugly code that is not in my opinion worth it, as the only breaking change in #1650 is for users of undocumented getErrors function as well as for the otherwise internal only errors annotation.

Might be worth releasing as patch version as I am not clear if we are following semver for undocumented functions, or in next major version, but is not worth releasing twice, once in an ugly version and then once in clean, at least in my opinion in terms of my unpaid time. Alas!

@yaacovCR yaacovCR closed this Jul 16, 2020
@yaacovCR yaacovCR deleted the fix-errors branch August 25, 2020 17:53
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.

1 participant

Comments