Skip to content

Exempt unresolvable federated entities from @EntityMapping checks #1225

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

Conversation

joshuapaulallen
Copy link
Contributor

Closes issue #1222. Updates check ensuring federated types have @EntityMapping to exempt entities with 'resolvable: false', since by definition those should not have an @EntityMapping.

Closes issue spring-projects#1222. Updates check ensuring federated types have
@EntityMapping to exempt entities with 'resolvable: false', since
by definition those should not have an @EntityMapping.

Signed-off-by: Josh Allen <[email protected]>
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label May 29, 2025
@joshuapaulallen joshuapaulallen marked this pull request as ready for review May 29, 2025 14:13
@rstoyanchev rstoyanchev self-assigned this May 29, 2025
@rstoyanchev rstoyanchev added type: regression A bug that is also a regression and removed status: waiting-for-triage An issue we've not yet triaged labels May 29, 2025
@rstoyanchev rstoyanchev added this to the 1.4.1 milestone May 29, 2025
@rstoyanchev rstoyanchev changed the title fix: exempt unresolvable federated entities from @EntityMapping check Exempt unresolvable federated entities from @EntityMapping checks May 29, 2025
rstoyanchev pushed a commit that referenced this pull request May 29, 2025
Update check ensuring federated types have @EntityMapping
to exempt entities with 'resolvable: false', since by definition
those should not have an @EntityMapping.

See gh-1225

Signed-off-by: Josh Allen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: regression A bug that is also a regression
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants