-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Preserve sources of variable values #3811
Open
yaacovCR
wants to merge
2
commits into
graphql:main
Choose a base branch
from
yaacovCR:variable-value-sources-rebased
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
✅ Deploy Preview for compassionate-pike-271cb3 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Hi @yaacovCR, I'm @github-actions bot happy to help you with this PR 👋 Supported commandsPlease post this commands in separate comments and only one per comment:
|
yaacovCR
force-pushed
the
variable-value-sources-rebased
branch
from
December 31, 2022 20:37
89c7b8d
to
a24779c
Compare
yaacovCR
force-pushed
the
variable-value-sources-rebased
branch
from
January 1, 2023 19:37
a24779c
to
9e6cf99
Compare
I guess this should technically be a BREAKING CHANGE as the signature changes for |
yaacovCR
added
PR: breaking change 💥
implementation requires increase of "major" version number
and removed
PR: internal 🏠
labels
Jan 12, 2023
mjmahone
reviewed
Jan 12, 2023
yaacovCR
force-pushed
the
variable-value-sources-rebased
branch
2 times, most recently
from
January 31, 2023 19:47
65145f9
to
ce33974
Compare
yaacovCR
force-pushed
the
variable-value-sources-rebased
branch
5 times, most recently
from
February 6, 2023 12:53
12f67c0
to
d8feffe
Compare
yaacovCR
force-pushed
the
variable-value-sources-rebased
branch
2 times, most recently
from
May 31, 2023 11:38
d8486ae
to
2211b7a
Compare
yaacovCR
force-pushed
the
variable-value-sources-rebased
branch
from
March 20, 2024 11:25
2211b7a
to
9f85bc3
Compare
yaacovCR
force-pushed
the
variable-value-sources-rebased
branch
from
September 15, 2024 10:37
9f85bc3
to
2906851
Compare
yaacovCR
force-pushed
the
variable-value-sources-rebased
branch
4 times, most recently
from
September 15, 2024 11:43
9dfe0a9
to
3d8be34
Compare
yaacovCR
force-pushed
the
variable-value-sources-rebased
branch
4 times, most recently
from
September 15, 2024 14:17
9e7a08b
to
12f30ec
Compare
yaacovCR
force-pushed
the
variable-value-sources-rebased
branch
5 times, most recently
from
September 17, 2024 12:21
5ee7a2f
to
2fe5343
Compare
Fixes graphql#3051 This change solves the problem of default values defined via SDL not always resolving correctly through introspection by preserving the original GraphQL literal in the schema definition. This changes argument and input field definitions `defaultValue` field from just the "value" to a new `GraphQLDefaultValueUsage` type which contains either -- but not both -- "value" and "literal" fields. Here is the flow for how a default value defined in an SDL would be converted into a functional schema and back to an SDL: **Before this change:** ``` (SDL) --parse-> (AST) --coerceInputLiteral--> (defaultValue config) --valueToAST--> (AST) --print --> (SDL) ``` `coerceInputLiteral` performs coercion which is a one-way function, and `valueToAST` is unsafe and set to be deprecated in graphql#3049. **After this change:** ``` (SDL) --parse-> (defaultValue literal config) --print --> (SDL) ```
By way of introducing type `VariableValues`, allows `getVariableValues` to return both the coerced values as well as the original sources, which are then made available in `ExecutionContext`.
yaacovCR
force-pushed
the
variable-value-sources-rebased
branch
from
September 17, 2024 13:56
2fe5343
to
55b9361
Compare
JoviDeCroock
approved these changes
Sep 18, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#3077 rebased on main.
Depends on #3810
@leebyron comments from original PR: