From 41eb2dff30a388e983800f607ac5bde967129df9 Mon Sep 17 00:00:00 2001
From: Santos Gallegos <stsewd@proton.me>
Date: Tue, 30 Jan 2024 11:40:37 -0500
Subject: [PATCH 1/2] Redirects: remove unused status field

---
 readthedocs/redirects/admin.py                   |  4 ++--
 .../migrations/0009_remove_status_field.py       | 16 ++++++++++++++++
 readthedocs/redirects/models.py                  |  3 ---
 3 files changed, 18 insertions(+), 5 deletions(-)
 create mode 100644 readthedocs/redirects/migrations/0009_remove_status_field.py

diff --git a/readthedocs/redirects/admin.py b/readthedocs/redirects/admin.py
index 399bcff828d..07972e69f69 100644
--- a/readthedocs/redirects/admin.py
+++ b/readthedocs/redirects/admin.py
@@ -7,9 +7,9 @@
 
 @admin.register(Redirect)
 class RedirectAdmin(admin.ModelAdmin):
-    list_display = ["project", "redirect_type", "from_url", "to_url", "status"]
+    list_display = ["project", "redirect_type", "from_url", "to_url", "position", "enabled"]
     list_select_related = ("project",)
-    list_filter = ("redirect_type", "status")
+    list_filter = ("redirect_type", "enabled")
     raw_id_fields = ("project",)
     search_fields = (
         "project__slug",
diff --git a/readthedocs/redirects/migrations/0009_remove_status_field.py b/readthedocs/redirects/migrations/0009_remove_status_field.py
new file mode 100644
index 00000000000..9eb8a52b8c9
--- /dev/null
+++ b/readthedocs/redirects/migrations/0009_remove_status_field.py
@@ -0,0 +1,16 @@
+# Generated by Django 4.2.9 on 2024-01-30 16:33
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("redirects", "0008_alter_redirect_position"),
+    ]
+
+    operations = [
+        migrations.RemoveField(
+            model_name="redirect",
+            name="status",
+        ),
+    ]
diff --git a/readthedocs/redirects/models.py b/readthedocs/redirects/models.py
index 445ee0db413..6e71740059b 100644
--- a/readthedocs/redirects/models.py
+++ b/readthedocs/redirects/models.py
@@ -116,9 +116,6 @@ class Redirect(models.Model):
         help_text=_("Order of execution of the redirect."),
     )
 
-    # TODO: remove this field and use `enabled` instead.
-    status = models.BooleanField(choices=[], default=True, null=True)
-
     create_dt = models.DateTimeField(auto_now_add=True)
     update_dt = models.DateTimeField(auto_now=True)
 

From 7bbe5792b55696638283f2587ae031d5da38e6b2 Mon Sep 17 00:00:00 2001
From: Santos Gallegos <stsewd@proton.me>
Date: Tue, 30 Jan 2024 11:42:09 -0500
Subject: [PATCH 2/2] Format

---
 readthedocs/redirects/admin.py | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/readthedocs/redirects/admin.py b/readthedocs/redirects/admin.py
index 07972e69f69..4a6ed9d8265 100644
--- a/readthedocs/redirects/admin.py
+++ b/readthedocs/redirects/admin.py
@@ -7,7 +7,14 @@
 
 @admin.register(Redirect)
 class RedirectAdmin(admin.ModelAdmin):
-    list_display = ["project", "redirect_type", "from_url", "to_url", "position", "enabled"]
+    list_display = [
+        "project",
+        "redirect_type",
+        "from_url",
+        "to_url",
+        "position",
+        "enabled",
+    ]
     list_select_related = ("project",)
     list_filter = ("redirect_type", "enabled")
     raw_id_fields = ("project",)