Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fixed Entity Framework Core integration steps and explained integration more thoroughly #51

Merged
merged 1 commit into from
Sep 10, 2023

Conversation

hexagonite
Copy link
Contributor

Hi, I tried to use this library in my project and I've found that the Entity Framework Core integration code snippet was missing parameterless constructors, as I got this exception during creation of migrations:

System.InvalidOperationException: "Cannot create an instance of value converter type 'UlidToStringConverter'. Ensure that the type can be instantiated and has a parameterless constructor, or use the overload of 'HasConversion' that accepts a delegate."

I used the same solution as here.

Moreover, I added further steps on how to use those ValueConverters so that the library is easy to use for experienced and inexperienced programmers alike.

Please let me know if there are any issues with my PR, I will gladly make all the necessary changes. I'm new to contributing to public repositories :)

@neuecc
Copy link
Member

neuecc commented Sep 10, 2023

Thank you! Very good!

@neuecc neuecc merged commit 6124723 into Cysharp:master Sep 10, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants