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.

Sign up for GitHub

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

Jump to bottom

Migrations: CHECK Required Columns on TPH Derived Types #2595

Open
Tracked by #22953
bricelam opened this issue Jul 10, 2015 · 3 comments
Open
Tracked by #22953

Migrations: CHECK Required Columns on TPH Derived Types #2595

bricelam opened this issue Jul 10, 2015 · 3 comments
Labels
area-relational-mapping punted-for-6.0 type-enhancement
Milestone
Backlog

Comments

@bricelam
Copy link
Contributor

For required properties on derived types, we could add a CHECK constraint to ensure it's NOT NULL when the discriminator is one that type or one of it's derived types.
@rowanmiller rowanmiller added this to the Backlog milestone Jul 10, 2015
@rowanmiller rowanmiller added the help wanted This issue involves technologies where we are not experts. Expert help would be appreciated. label Jul 10, 2015
@bricelam bricelam mentioned this issue Jan 15, 2019
Closed
@ajcvickers ajcvickers mentioned this issue Mar 22, 2019
Closed
@divega divega added good first issue This issue should be relatively straightforward to fix. help wanted This issue involves technologies where we are not experts. Expert help would be appreciated. and removed help wanted This issue involves technologies where we are not experts. Expert help would be appreciated. good first issue This issue should be relatively straightforward to fix. labels May 31, 2019
@ajcvickers ajcvickers added good first issue This issue should be relatively straightforward to fix. and removed help wanted This issue involves technologies where we are not experts. Expert help would be appreciated. labels Aug 5, 2019
@bricelam bricelam mentioned this issue May 14, 2020
Closed
@roji roji removed this from the Backlog milestone Sep 14, 2020
@roji
Copy link
Member

roji commented Sep 14, 2020

Clearing milestone to determine whether we want this to be in EF Core or in EFCore.CheckConstraints (efcore/EFCore.CheckConstraints#6)

@bricelam
Copy link
Contributor Author

Also useful for optional dependents using table splitting

@ajcvickers ajcvickers added this to the Backlog milestone Sep 18, 2020
@ajcvickers ajcvickers mentioned this issue Oct 11, 2020
Open
63 tasks
@ajcvickers ajcvickers modified the milestones: Backlog, 6.0.0 Oct 28, 2020
@bricelam
Copy link
Contributor Author

bricelam commented Nov 4, 2020

A Stack Overflow question about this: EF Core saves null value of required property

@ajcvickers ajcvickers modified the milestones: 6.0.0, Backlog May 5, 2021
@bricelam bricelam mentioned this issue Jul 29, 2021
Open
@AndriySvyryd AndriySvyryd mentioned this issue Oct 19, 2022
Open
@bricelam bricelam removed their assignment Jul 8, 2023
@ajcvickers ajcvickers removed the good first issue This issue should be relatively straightforward to fix. label May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area-relational-mapping punted-for-6.0 type-enhancement
Projects
None yet
Milestone
Backlog
Development

No branches or pull requests
6 participants
@bricelam @divega @ajcvickers @roji @rowanmiller @AndriySvyryd