-
Notifications
You must be signed in to change notification settings - Fork 13
Fixes on Demand Control #1027
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
base: main
Are you sure you want to change the base?
Fixes on Demand Control #1027
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
🚀 Snapshot Release (
|
Package | Version | Info |
---|---|---|
@graphql-tools/batch-delegate |
9.0.36-alpha-3fb8d587b41638f428d90309c8cf2dcd31edc8e3 |
npm ↗︎ unpkg ↗︎ |
@graphql-tools/batch-execute |
9.0.16-alpha-3fb8d587b41638f428d90309c8cf2dcd31edc8e3 |
npm ↗︎ unpkg ↗︎ |
@graphql-tools/delegate |
10.2.18-alpha-3fb8d587b41638f428d90309c8cf2dcd31edc8e3 |
npm ↗︎ unpkg ↗︎ |
@graphql-tools/federation |
3.2.2-alpha-3fb8d587b41638f428d90309c8cf2dcd31edc8e3 |
npm ↗︎ unpkg ↗︎ |
@graphql-mesh/fusion-runtime |
0.11.11-alpha-3fb8d587b41638f428d90309c8cf2dcd31edc8e3 |
npm ↗︎ unpkg ↗︎ |
@graphql-hive/gateway |
1.13.7-alpha-3fb8d587b41638f428d90309c8cf2dcd31edc8e3 |
npm ↗︎ unpkg ↗︎ |
@graphql-hive/nestjs |
1.0.11-alpha-3fb8d587b41638f428d90309c8cf2dcd31edc8e3 |
npm ↗︎ unpkg ↗︎ |
@graphql-hive/plugin-aws-sigv4 |
1.0.8-alpha-3fb8d587b41638f428d90309c8cf2dcd31edc8e3 |
npm ↗︎ unpkg ↗︎ |
@graphql-mesh/plugin-opentelemetry |
1.3.55-alpha-3fb8d587b41638f428d90309c8cf2dcd31edc8e3 |
npm ↗︎ unpkg ↗︎ |
@graphql-mesh/plugin-prometheus |
1.3.43-alpha-3fb8d587b41638f428d90309c8cf2dcd31edc8e3 |
npm ↗︎ unpkg ↗︎ |
@graphql-hive/gateway-runtime |
1.8.1-alpha-3fb8d587b41638f428d90309c8cf2dcd31edc8e3 |
npm ↗︎ unpkg ↗︎ |
@graphql-tools/stitch |
9.4.23-alpha-3fb8d587b41638f428d90309c8cf2dcd31edc8e3 |
npm ↗︎ unpkg ↗︎ |
@graphql-tools/stitching-directives |
3.1.33-alpha-3fb8d587b41638f428d90309c8cf2dcd31edc8e3 |
npm ↗︎ unpkg ↗︎ |
@graphql-tools/wrap |
10.0.36-alpha-3fb8d587b41638f428d90309c8cf2dcd31edc8e3 |
npm ↗︎ unpkg ↗︎ |
🚀 Snapshot Release (Node Docker Image)The latest changes of this PR are available as image on GitHub Container Registry (based on the declared
|
🚀 Snapshot Release (Bun Docker Image)The latest changes of this PR are available as image on GitHub Container Registry (based on the declared
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
f8c57ab
to
3be8c97
Compare
3be8c97
to
f87a0b3
Compare
00a4131
to
80c5306
Compare
Co-authored-by: Copilot <[email protected]>
80c5306
to
3fb8d58
Compare
Ref GW-300
includeExtensionMetadata
,cost.esTimated
should always be added to the result extensions even if no cost is calculated. So the estimated cost will be 0 in this case. For example{ __typename }
never hits any subgraph or QP, then it should be 0.