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

v1: Entity Framework migration #963

Open
wants to merge 1 commit into
base: develop-v1
Choose a base branch
from
Open

v1: Entity Framework migration #963

wants to merge 1 commit into from

Conversation

rasmus
Copy link
Member

@rasmus rasmus commented Nov 11, 2022

No description provided.

@github-actions
Copy link

github-actions bot commented Apr 8, 2023

Hello there!

We hope this message finds you well. We wanted to let you know that we have noticed that there has been
no activity on this pull request for the past 90 days, which makes it a stale pull request.

As a result, we will be closing this pull request within the next seven days. If you still
think this pull request is necessary or relevant, please feel free to update it or leave a
comment within the next seven days.

Thank you for your contributions and understanding.

Best regards,
EventFlow

@kyle-bradley
Copy link
Contributor

@rasmus I migrated EntityFramework on my fork. Let me know if you are happy with my approach and I'll happily do a PR here. I currently only tested PostgreSql & InMemory so would also need to test out MsSQL & SQLite
kyle-bradley#17

@rasmus
Copy link
Member Author

rasmus commented Oct 18, 2024

@kyle-bradley that would be awesome!

@rasmus
Copy link
Member Author

rasmus commented Dec 24, 2024

@kyle-bradley would you be interested in opening a PR with your migration of EF for v1?

@rasmus
Copy link
Member Author

rasmus commented Dec 24, 2024

Or are the changes I did here on the right track? I'm not a EF user and thus have little to no experience working with it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants