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

Allow DeleteBehavior to be reset to the default #15295

Open
Tracked by #22952
ajcvickers opened this issue Apr 9, 2019 · 1 comment
Open
Tracked by #22952

Allow DeleteBehavior to be reset to the default #15295

ajcvickers opened this issue Apr 9, 2019 · 1 comment
Labels
area-model-building good first issue This issue should be relatively straightforward to fix. type-enhancement
Milestone

Comments

@ajcvickers
Copy link
Member

Breaking out of #12661

For example, by allowing null to be set in the fluent API.

Note that the value in core metadata is not nullable, so this means either making it nullable as well (which would be a more significant break) or causing null in the fluent API to re-run the convention logic used if no value has been set.

@AndriySvyryd
Copy link
Member

Note that setting it to null will just reset the value to the model default, it won't rerun the conventions that might have modified it. To achieve that we would either need default value conventions or metadata layering (part of #214) where higher configuration source would not remove the lower source value, but just hide it.

@ajcvickers ajcvickers added this to the Backlog milestone Apr 12, 2019
@ajcvickers ajcvickers added the good first issue This issue should be relatively straightforward to fix. label Sep 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-model-building good first issue This issue should be relatively straightforward to fix. type-enhancement
Projects
None yet
Development

No branches or pull requests

2 participants