apollographql / federation

🌐  Build and scale a single data graph across multiple services with Apollo's federation gateway.
https://apollographql.com/docs/federation/
Other
668 stars 254 forks source link

Relax external field requirements #3142

Closed clenfest closed 2 months ago

clenfest commented 2 months ago

Relax error for detecting whether @external field matches. If the external field has extra optional parameters, allow it to pass composition as being able to pass the parameter is not necessary for a required field.

changeset-bot[bot] commented 2 months ago

🦋 Changeset detected

Latest commit: 5c7066804eaee2b9ffd6ce831b68575cd3d4a78b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 7 packages | Name | Type | | --------------------------------------- | ----- | | @apollo/composition | Patch | | @apollo/gateway | Patch | | @apollo/federation-internals | Patch | | @apollo/query-planner | Patch | | @apollo/query-graphs | Patch | | @apollo/subgraph | Patch | | apollo-federation-integration-testsuite | Patch |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

netlify[bot] commented 2 months ago

Deploy Preview for apollo-federation-docs canceled.

Name Link
Latest commit 5c7066804eaee2b9ffd6ce831b68575cd3d4a78b
Latest deploy log https://app.netlify.com/sites/apollo-federation-docs/deploys/66e1e5867372b800088cd679
codesandbox-ci[bot] commented 2 months ago

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.