Summary

Switch from index_together to Meta.Indexes for ReviewRequestVisit.

Review Request #14591 — Created Sept. 4, 2025 and submitted Sept. 16, 2025, 7:33 a.m.

Owner

Repository

Review Board

Branch

master

Bugs

Depends On

Groups

People

Description

index_together is deprecated and will be removed in Django 5.1. This
evolution changes ReviewRequestVisit to use Meta.indexes instead.

Testing Done

Ran evolution.

Commits

Summary	ID
Switch from index_together to Meta.Indexes for ReviewRequestVisit. `index_together` is deprecated and will be removed in Django 5.1. This evolution changes ReviewRequestVisit to use `Meta.indexes` instead. Testing Done: Ran evolution.	uorlrvzwzousmslrrmsqrtnotsxnztxt

Issues

Description	From	Last Updated
Upgrades will also need to be tried on MySQL and Postgres.	chipx86	Sept. 4, 2025, 1:58 p.m.

flake8 passed.

JSHint passed.

Fix it!

An issue was opened. Show all issues

Upgrades will also need to be tried on MySQL and Postgres.

That's a lot of infrastructure to set up. Isn't this why django-evolution has a huge test suite?

It is, but I just wanted to check because this would be the first evolution doing this in our codebase.

I can do the test if you want, but you'll need to remind me. This shoud be easy with the Docker images.