-
Notifications
You must be signed in to change notification settings - Fork 58
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
Migrate to Postgresql #408
Comments
Do you want help with this? |
@opeolluwa Yes, but I still haven’t decided what to use for PostgreSQL. I’m leaning more towards Diesel over SeaORM—it seems simpler and doesn’t clutter the code with ORM-related elements everywhere. I also need to put more effort into my #458 PR to make it more stable so that others can contribute, as it’s currently a bit messy. |
yes, I've used both in production environment. Also on the other end, Diesel isn't async-first unlike, SeaORM. |
Do you want me to look at this |
@opeolluwa I’m currently using diesel-async for async support, and it seems to be working correctly. If you’re interested in continuing the work, that would be fantastic. I haven’t had much time lately to make progress, so any help would be greatly appreciated! |
I'll see what can be done |
@ndelvalle do you want to use diese-async or I should revamp with sea-orm |
@opeolluwa i think using async diesel is the best idea. |
alright |
I'll put it up locally tomorrow! |
@ndelvalle would you be kind to tag the branch you're working on |
I just added you as a collaborator, feel free to keep working on the same branch or tag it. |
awesome thanks |
@ndelvalle the use-diesel branch has a handful of issues, I'm going to squash it and incrementally make PR to the main branch till all it is all setup |
Yeah @opeolluwa , we don't need to use the use-diesel branch, you can create another one and we can work on that one. |
Awesome! I've done that |
After doing some research I think I will give https://github.com/SeaQL/sea-orm a try and implement it.
Interesting reads about different ORM options:
The text was updated successfully, but these errors were encountered: