-
Notifications
You must be signed in to change notification settings - Fork 405
Django 5.1 Support - deprecated index_together #567
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: develop
Are you sure you want to change the base?
Conversation
Django 5.1 ... index_together deprecated.
Django 5.1 .... index_together deprecated
|
Waiting for this pull request to be merged soon |
|
Any updates on when this could be merged? @llazzaro |
|
Seems that @llazzaro is not responding. Or has anybody else infomation about new releases, maybe @jdufresne ? |
|
I will try to take a look today |
@llazzaro any updates? I know that there is a django5_scheduler package in pypi available, but this is pinned to django5 which is not really good when you have to support django4 and django5 LTS versions |
|
Hi @llazzaro , just another reminder. What is the status. I know there is already a package available for django5 https://github.com/NCIAdmin/django5-scheduler, but this package sticks to django5 which is not working if you have to support django 4.2 LTS and django5.2 LTS. The changes in this pull request will support 4.2 and 5.2 (already tested), I would prefer to use the package from you (@llazzaro ) instead of depeding on forks (by the way the django5-scheduler maintainer is also inkognito mode and does not provide any contact options) |
|
This is actually missing the migration, you can add it from here: # Generated by Django 4.2.19 on 2025-08-21 18:48
from django.db import migrations
class Migration(migrations.Migration):
dependencies = [
('schedule', '0014_use_autofields_for_pk'),
]
operations = [
migrations.RenameIndex(
model_name='calendarrelation',
new_name='schedule_ca_content_cddadb_idx',
old_fields=('content_type', 'object_id'),
),
migrations.RenameIndex(
model_name='event',
new_name='schedule_ev_start_a258cb_idx',
old_fields=('start', 'end'),
),
migrations.RenameIndex(
model_name='eventrelation',
new_name='schedule_ev_content_6ceecb_idx',
old_fields=('content_type', 'object_id'),
),
migrations.RenameIndex(
model_name='occurrence',
new_name='schedule_oc_start_76a2f8_idx',
old_fields=('start', 'end'),
),
] |
|
Thought I'd try my luck pinging @llazzaro and @jdufresne before I try and replace the dependency, or fork it. This PR seems small enough that coupled with the migration above it should hopefully be a tick and flick. Although for others on this thread I am going to try https://github.com/NCIAdmin/django5-scheduler first |
Fix pulled from Django 4.2 release notes here -
https://docs.djangoproject.com/en/5.1/releases/4.2/#deprecated-features-4-2
Migrating existing index_together should be handled as a migration. For example:
Should become: