Skip to content
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

feat: allow STI child entities to have non-nullable relationships #10155

Open
wants to merge 1 commit into
base: 3.2.x
Choose a base branch
from

Conversation

simPod
Copy link
Contributor

@simPod simPod commented Oct 19, 2022

Currently, child entities cannot have relationships marked as non-nullable because SchemaTool generates the relationship join column as NOT NULL.

It has to be marked as nullable, the same as is done for non-nullable columns that are present only in child entity.

@simPod simPod force-pushed the sti branch 2 times, most recently from a6ffb28 to 459649e Compare October 19, 2022 12:53
@simPod simPod marked this pull request as ready for review October 19, 2022 12:53
@mpdude
Copy link
Contributor

mpdude commented Jan 25, 2023

Do you mean

Currently, child entities cannot have relationships marked as nullable…?

@simPod
Copy link
Contributor Author

simPod commented Jan 25, 2023

@simPod simPod changed the base branch from 2.14.x to 2.15.x June 23, 2023 06:56
@simPod simPod changed the base branch from 2.15.x to 2.16.x August 1, 2023 18:50
@simPod simPod changed the base branch from 2.16.x to 2.19.x March 4, 2024 09:41
@simPod simPod changed the base branch from 2.19.x to 3.2.x May 27, 2024 06:19
Copy link
Contributor

There hasn't been any activity on this pull request in the past 90 days, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 7 days.
If you want to continue working on it, please leave a comment.

@github-actions github-actions bot added the Stale label Jan 10, 2025
@simPod
Copy link
Contributor Author

simPod commented Jan 10, 2025 via email

@github-actions github-actions bot removed the Stale label Jan 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants