Skip to content

Backport v16 commits #4388

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.

Sign up for GitHub

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

Closed
wants to merge 21 commits into from
Closed

Backport v16 commits #4388

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
434cc98
Add redirect for /api (#4327)
JoviDeCroock Jan 15, 2025
beb9fa7
Increase print/visit performance (#4312)
JoviDeCroock May 4, 2025
d5c1e8d
fix eslint errors with website
yaacovCR Dec 3, 2024
26c7de8
fix sidebar for documentation and `/api-v16` (#4331)
dimaMachina Jan 29, 2025
31a88a3
Add cspell exception (#4335)
JoviDeCroock Jan 29, 2025
5c18922
Improve flow of documentation around GraphiQL (#4340)
benjie Feb 5, 2025
466e16c
typofix: removes extra parenthesis from getting started code snippet …
rabahalishah Feb 12, 2025
ac7ad7a
First draft for upgrade guide to v17 (#4310)
JoviDeCroock Feb 19, 2025
1d0cc8f
docs(getting-started): promises current links (#4352)
guspan-tanadi Mar 4, 2025
3805bea
fixed wrong variable name (#4351)
fto-dev Mar 8, 2025
5a169fe
fix(coerce-input-value): input object coercion rejects arrays (#4367)
JoviDeCroock May 4, 2025
b3879c8
feat(execution): add max coercion errors option to execution context …
cristunaranjo Apr 10, 2025
7662a00
Correct some syntax (#4369)
JoviDeCroock Apr 10, 2025
0f4a254
Update docs for execution options (#4368)
JoviDeCroock Apr 10, 2025
e00e83a
docs: Update getting-started.mdx (#4373)
Shubhdeep12 Apr 24, 2025
dd6f507
Refactor every code-first example to leverage resolve (#4372)
JoviDeCroock Apr 24, 2025
eb41fba
Change to gqlConf 2025 (#4378)
JoviDeCroock Apr 25, 2025
85e5e11
Add missing parenthesis (#4379)
benjie Apr 25, 2025
8b73a88
docs: anatomy of a resolver (#4381)
sarahxsanders May 3, 2025
28ace28
docs: understanding graphql errors (#4382)
sarahxsanders May 3, 2025
5460ae5
Fix tests
JoviDeCroock May 4, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 1 addition & 2 deletions eslint.config.mjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,7 @@ export default tsConfig(
'npmDist',
'npmEsmDist',
'denoDist',
'website/.next',
'website/out',
'website',
'integrationTests/ts/*.ts',
],
},
Expand Down
2 changes: 1 addition & 1 deletion src/execution/__tests__/executor-test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1441,7 +1441,7 @@ describe('Execute: Handles basic execution tasks', () => {
errors: [
{
message:
'Variable "$data" got invalid value { email: "", wrongArg: "wrong", wrongArg2: "wrong", wrongArg3: "wrong" }; Field "wrongArg" is not defined by type "User".',
'Variable "$data" has invalid value: Expected value of type "User" not to include unknown field "wrongArg", found: { email: "", wrongArg: "wrong", wrongArg2: "wrong", wrongArg3: "wrong" }.',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this text fix should be integrated into the individual cherry-picked PR so that all tests pass on git bisect when we merge this PR with the rebase strategy.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

in particular looks like this commit could be integrated into the backporting of #4366

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i set up a PR with what I mean in #4413 => i also port the changes in coerceInputValue from #4367 to validateInputValue

locations: [{ line: 2, column: 17 }],
},
{
Expand Down
Loading