-
-
Notifications
You must be signed in to change notification settings - Fork 48
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
Add Django-5.1 support #152 #153
Conversation
162ae3f
to
0ae2c48
Compare
@@ -216,7 +216,6 @@ def set_up_test_model( | |||
# Make the "current" state. | |||
model_options = { | |||
'swappable': 'TEST_SWAP_MODEL', | |||
'index_together': [['weight', 'pink']] if index_together else [], |
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.
index_together
is deprecated (dj42) and removed (dj51).
https://docs.djangoproject.com/en/5.1/releases/5.1/#features-removed-in-5-1
Redshift ignores indexes, so django-redshift-backend does not create any indexes.
I removed this definition because I left it in place when I brought it from the original Django test code.
0ae2c48
to
5bc006a
Compare
This comment has been moved into issue #154 |
5bc006a
to
1f95c66
Compare
backward migration is not worked.
|
1f95c66
to
b5e7f6a
Compare
fixed by #156 |
Tasks