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

fix: don't define the @removeNullVariables directive in the base schema #577

Merged
merged 1 commit into from
Aug 18, 2023

Conversation

fiam
Copy link
Contributor

@fiam fiam commented Aug 18, 2023

Support it, but require the caller to declare it themselves if they want
to use it.

References wundergraph/wundergraph#985

Also, remove a couple of deprecated usages of ioutil

Support it, but require the caller to declare it themselves if they want
to use it.

References wundergraph/wundergraph#985

Also, remove a couple of deprecated usages of ioutil
@fiam fiam merged commit 6239b61 into master Aug 18, 2023
6 checks passed
@fiam fiam deleted the alberto/eng-2331-duplicate-removenullvariables branch August 18, 2023 13:35
devsergiy pushed a commit that referenced this pull request Aug 29, 2023
…ma (#577)

Support it, but require the caller to declare it themselves if they want
to use it.

References wundergraph/wundergraph#985

Also, remove a couple of deprecated usages of ioutil
devsergiy added a commit that referenced this pull request Aug 29, 2023
pvormste pushed a commit to TykTechnologies/graphql-go-tools that referenced this pull request Oct 6, 2023
…ma (wundergraph#577)

Support it, but require the caller to declare it themselves if they want
to use it.

References wundergraph/wundergraph#985

Also, remove a couple of deprecated usages of ioutil
pvormste added a commit to TykTechnologies/graphql-go-tools that referenced this pull request Oct 6, 2023
#389)

…ma (wundergraph#577)

Support it, but require the caller to declare it themselves if they want
to use it.

References wundergraph/wundergraph#985

Also, remove a couple of deprecated usages of ioutil

Co-authored-by: Alberto García Hierro <alberto@garciahierro.com>
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.

None yet

3 participants