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.

Sign up for GitHub

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

Jump to bottom

Query: Convert enum to underlying CLR type in type mapping when type … #25793

Merged
merged 1 commit into from
Sep 1, 2021
Merged

Query: Convert enum to underlying CLR type in type mapping when type … #25793

merged 1 commit into from
Sep 1, 2021

Conversation

smitpatel
Copy link
Contributor

…mapping is not for enum

Resolves #21770

@smitpatel smitpatel requested a review from a team August 30, 2021 21:11
@smitpatel
Copy link
Contributor Author

@lauxjpn @roji - Added tests pass without fix for all built-in providers. It would be useful if you can verify the fix on your end too.

@roji
Copy link
Member

roji commented Sep 1, 2021

@smitpatel tests pass for me on Npgsql 👍

@smitpatel smitpatel merged commit 6c8eaa0 into release/6.0 Sep 1, 2021
@smitpatel smitpatel deleted the smit/zbb30-2 branch September 1, 2021 21:36
@ajcvickers ajcvickers mentioned this pull request Sep 9, 2021
Closed
@clement128
Copy link

Can this also merge to efcore 5.0? thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@roji roji roji approved these changes

@AndriySvyryd AndriySvyryd AndriySvyryd approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@smitpatel @roji @clement128 @AndriySvyryd