Skip to content

Comments

[pull] 16.x.x from graphql:16.x.x#65

Merged
pull[bot] merged 3 commits intoehtick:16.x.xfrom
graphql:16.x.x
Jul 14, 2025
Merged

[pull] 16.x.x from graphql:16.x.x#65
pull[bot] merged 3 commits intoehtick:16.x.xfrom
graphql:16.x.x

Conversation

@pull
Copy link

@pull pull bot commented Jul 1, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.2)

Can you help keep this open source service alive? 💖 Please sponsor : )

@pull pull bot locked and limited conversation to collaborators Jul 1, 2025
@pull pull bot added the ⤵️ pull label Jul 1, 2025
JoviDeCroock and others added 2 commits July 7, 2025 17:36
Supersedes #4445
Fixes #4443

`VariablesInAllowedPositionRule` already validates this
)

The example shows graphql implementation, where the "root" is by mistake
put as string as part of the schema definition instead being a native js
const

BEFORE (root is undefined and wongly put into the schema itself):


![image](https://github.com/user-attachments/assets/3908e84e-1c3e-43b5-8633-163ea66fdfa0)


AFTER (root correctly defined outside of the schema):


![image](https://github.com/user-attachments/assets/e14d51cc-2f36-450d-aea8-384d6c6a0703)
@pull pull bot merged commit 8dc2955 into ehtick:16.x.x Jul 14, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants