-
Notifications
You must be signed in to change notification settings - Fork 10
[WIP] Introduce django #79
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
base: master
Are you sure you want to change the base?
Conversation
|
The application is in a running state, all features are brought to the django app. Also, there are probably a number of places which do not follow django best practices - I'd suggest to get the application into proper state for release, and shape the model and surrounding code on the go Before merge to master, I'd also suggest re-generating all migrations, there is no need to keep the initial micro-migrations separate |
|
I'm wondering how best to support existing installations. Maybe slam the existing sql code into the initial migration so that when Django applies it on an existing database, it's basically a no-op, then go from there? 🤔 |
|
Django has a tool to deal with existing databases, basically generating the initial model and preparing it for the db migrations: https://docs.djangoproject.com/en/5.2/howto/legacy-databases/ |
|
@liquid-metal are you still working on this? the PR would still need a lot of cleanup before merging (for instance, code that is moved should be |
As discussed in #78, we give django a shot - brief comparison with sqlalchemy will follow, though I am already a bit biased :)
This PR tracks my dev branch on the topic for reference, I will remove the draft state when we can think about actually merging. Until then, there will be a million TODO comments