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 complex type mappings to the default relational mappings #32867

Merged
merged 2 commits into from
Jan 23, 2024

Conversation

ajcvickers
Copy link
Contributor

Replaces #32816
Fixes #32699

As discussed, leaving FromSql to use the default mappings, but add complex types to the default mappings.

@ajcvickers ajcvickers requested a review from a team January 19, 2024 16:33
Replaces #32816
Fixes #32699

As discussed, leaving FromSql to use the default mappings, but add complex types to the default mappings.
@ajcvickers ajcvickers force-pushed the 240118_RiddlesInTheDark branch from 81b7a40 to 3fdb4d0 Compare January 23, 2024 10:49
@ajcvickers
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ajcvickers ajcvickers merged commit 32886fb into main Jan 23, 2024
7 checks passed
@ajcvickers ajcvickers deleted the 240118_RiddlesInTheDark branch January 23, 2024 13:07
ajcvickers added a commit that referenced this pull request Jan 23, 2024
* Add complex type mappings to the default relational mappings

Replaces #32816
Fixes #32699

As discussed, leaving FromSql to use the default mappings, but add complex types to the default mappings.

* Add note to SelectExpression
ajcvickers added a commit that referenced this pull request Jan 23, 2024
* Add complex type mappings to the default relational mappings

Replaces #32816
Fixes #32699

As discussed, leaving FromSql to use the default mappings, but add complex types to the default mappings.

* Add note to SelectExpression
maumar pushed a commit that referenced this pull request Feb 7, 2024
…pings (#32901)

* Add complex type mappings to the default relational mappings (#32867)

* Add complex type mappings to the default relational mappings

Replaces #32816
Fixes #32699

As discussed, leaving FromSql to use the default mappings, but add complex types to the default mappings.

* Add note to SelectExpression

* Fix merge and add quirks
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.

Exception when using FromSql on entity with ComplexProperty
2 participants