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

Model building: Relationships docs page is too complex and long #489

Closed
divega opened this issue Oct 6, 2017 · 0 comments · Fixed by #1936
Closed

Model building: Relationships docs page is too complex and long #489

divega opened this issue Oct 6, 2017 · 0 comments · Fixed by #1936

Comments

@divega
Copy link
Contributor

divega commented Oct 6, 2017

Structure is mixed in form of how to configure type of relationship & type of configuration.

Also, is the information about what happens with multiple navigation pairs (then convention won't discover relationship and you need to configure them explicitly) good enough? See the first warning in https://docs.microsoft.com/ef/core/modeling/relationships.

Originally from https://github.com/aspnet/Release/issues/189 and #424.

@divega divega changed the title Structure of relationships page is mixed in form of how to configure type of relationship & type of configuration Model building: Relationships docs page is too complex and long Oct 6, 2017
@ajcvickers ajcvickers self-assigned this Oct 6, 2017
@ajcvickers ajcvickers added this to the 2.1.0 milestone Oct 6, 2017
@divega divega modified the milestones: 2.1.0, 2.2.0 Jul 10, 2018
@divega divega modified the milestones: 2.2.0, 3.0.0 Nov 10, 2018
@ajcvickers ajcvickers modified the milestones: 3.0.0, Backlog Aug 28, 2019
@ajcvickers ajcvickers modified the milestones: Backlog, 3.1.0 Sep 6, 2019
AndriySvyryd added a commit that referenced this issue Nov 21, 2019
AndriySvyryd added a commit that referenced this issue Nov 21, 2019
AndriySvyryd added a commit that referenced this issue Nov 21, 2019
AndriySvyryd added a commit that referenced this issue Nov 21, 2019
Fixes #489
Fixes #1307
Fixes #1315
Fixes #1346
Fixes #1617
Fixes #1658
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants