Skip to content

Commit

Permalink
Change to major because it may break existing use cases
Browse files Browse the repository at this point in the history
  • Loading branch information
eddeee888 committed Dec 16, 2024
1 parent c932044 commit bab88db
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions .changeset/thick-pianos-smoke.md
Original file line number Diff line number Diff line change
@@ -1,9 +1,11 @@
---
'@graphql-codegen/visitor-plugin-common': patch
'@graphql-codegen/typescript-resolvers': patch
'@graphql-codegen/plugin-helpers': patch
'@graphql-codegen/visitor-plugin-common': major
'@graphql-codegen/typescript-resolvers': major
'@graphql-codegen/plugin-helpers': major
---

Fix `mappers` usage with Federation

`mappers` was previously used as `__resolveReference`'s first param (usually called "reference"). However, this is incorrect because `reference` interface comes directly from `@key` and `@requires` directives. This patch fixes the issue by creating a new `FederationTypes` type and use it as the base for federation entity types when being used to type entity references
`mappers` was previously used as `__resolveReference`'s first param (usually called "reference"). However, this is incorrect because `reference` interface comes directly from `@key` and `@requires` directives. This patch fixes the issue by creating a new `FederationTypes` type and use it as the base for federation entity types when being used to type entity references.

BREAKING CHANGES: No longer generate `UnwrappedObject` utility type, as this was used to support the wrong previously generated type.

0 comments on commit bab88db

Please sign in to comment.