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

Metadata: Allow unbound navigation properties #2140

Closed
bricelam opened this issue May 7, 2015 · 7 comments
Closed

Metadata: Allow unbound navigation properties #2140

bricelam opened this issue May 7, 2015 · 7 comments
Assignees
Labels
closed-fixed The issue has been fixed and is/will be included in the release indicated by the issue milestone. type-enhancement
Milestone

Comments

@bricelam
Copy link
Contributor

bricelam commented May 7, 2015

For example:

mb.Entity<Post>().Reference<Blog>("Blog").InverseCollection("Posts");

We could then store this in model snapshot and use it to help detect foreign key column renames.

It could also make configuring foreign keys without any navigation properties a little less awkward.

@rowanmiller rowanmiller added this to the 7.0.0 milestone May 11, 2015
@rowanmiller
Copy link
Contributor

@smitpatel please talk to @ajcvickers about this one

@divega
Copy link
Contributor

divega commented Jul 23, 2015

@bricelam @rowanmiller @ajcvickers @AndriySvyryd @smitpatel is this item missing a priority or do we need to investigate it further? @lajones and I talked about it today because it would allow cleaning up reveng and our provider contract for it.

@ajcvickers
Copy link
Member

@bricelam @rowanmiller @AndriySvyryd @smitpatel @lajones I suspect that Migrations doesn't need it yet because its not diffing navigation properties, but @bricelam can confirm. This makes it lower priority for Migrations, so the priority may depend on its importance for reverse engineering.

@divega
Copy link
Contributor

divega commented Jul 24, 2015

Unassigning and moving to backlog. We can make migrations smarter and tidy up reverse engineering if we add this later.

@divega
Copy link
Contributor

divega commented Nov 4, 2015

Clearing up the milestone as we mentioned in a recent reverse engineering discussion that we may decide to implement this for RTM.

@rowanmiller
Copy link
Contributor

Initially this should be similar to shadow entities where they are non-functional

@smitpatel
Copy link
Member

Filed #5154 & #5155 for RevEng & Migrations to utilize shadow navigations.

@ajcvickers ajcvickers added the closed-fixed The issue has been fixed and is/will be included in the release indicated by the issue milestone. label Oct 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed-fixed The issue has been fixed and is/will be included in the release indicated by the issue milestone. type-enhancement
Projects
None yet
Development

No branches or pull requests

5 participants