-
Notifications
You must be signed in to change notification settings - Fork 6
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
Upgrade to django 4.2 #1217
Upgrade to django 4.2 #1217
Conversation
…rectory-api into Upgrade-to-Django-4.2 Merging branch
@@ -0,0 +1,24 @@ | |||
# Generated by Django 4.1.13 on 2024-01-15 11:12 | |||
|
|||
from django.db import migrations, models |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are there migrations in a Django bump?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added by Bobby - so cannot comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @sebs-code, somewhere along the line the models had been adjusted without migration files being created. They don't need to be included in this release but will need to be added at some point.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @bobby-didcoding
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They might as well go in now... if they have to at some point!
Upgrade to django 4.2
Workflow
Merging