Closed sachindshinde closed 2 months ago
Latest commit: 2e54f9873821ebc144be129bb0c816f356b1f142
The changes in this PR will be included in the next version bump.
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
Name | Link |
---|---|
Latest commit | 2e54f9873821ebc144be129bb0c816f356b1f142 |
Latest deploy log | https://app.netlify.com/sites/apollo-federation-docs/deploys/66e4c9aa208dec0008b83263 |
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.
While reviewing recent code changes that released in
2.9.0
, I noticed a few bugs in the code, so I've pushed some fixes for them in this PR (along with some fixes for more ancient bugs).I'll comment on the code with more details, but to summarize the fixes:
null
s/undefined
s better.SUM
strategy, as using it currently wouldn't yield expected results since directive applications are deduped before being merged.@link
applications to the supergraph schema for supergraph specs needed by subgraph directives has been updated to handle specs that have multiple directives that need composing (the behavior before was adding spurious directive definitions to the supergraph schema).@core
/@link
was off.