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

Add documentation for TPT configuration. #2722

Merged
merged 3 commits into from
Oct 2, 2020
Merged

Add documentation for TPT configuration. #2722

merged 3 commits into from
Oct 2, 2020

Conversation

AndriySvyryd
Copy link
Member

Fixes #1979

> [!NOTE]
> If the primary key constraint is renamed the new name will be applied to all tables mapped to the hierarchy, future EF versions will allow renaming the constraint only for a particular table.

If you are employing bulk configuration you can retrieve the column name for a specific table by calling `GetColumnType`.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

column name from GetColumnType?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does it take parameter? if yes then how to get the value of parameter.
Once we had API docs updated, may be we can just reference actual API.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The snippet below answers all your questions.
I tried linking, but it seems only types can be linked using xref

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Snippet does not use GetColumnType API.

xref can be used for any API which has page in .NET API explorer. What are you trying to link? GetColumnName overload which takes StoreObjectIdentifier would be missing in API explorer for now.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've tried linking the old GetColumnName - xref:Microsoft.EntityFrameworkCore.RelationalPropertyExtensions.GetColumnName

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Linked it to "all overload" page which works. In future we should update it to specific API.

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.

Docs for relational inheritance mapping
2 participants