Simplify Segment version field with default value

Author: emyller

api/segments/migrations/0030_add_default_to_segment_version.py

@@ -0,0 +1,23 @@
+# Generated by Django 4.2.22 on 2025-11-11 00:08
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("segments", "0029_add_is_system_segment"),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name="historicalsegment",
+            name="version",
+            field=models.IntegerField(default=1, null=True),
+        ),
+        migrations.AlterField(
+            model_name="segment",
+            name="version",
+            field=models.IntegerField(default=1, null=True),
+        ),
+    ]

api/segments/models.py

@@ -87,8 +87,7 @@ class Segment(
         Feature, on_delete=models.CASCADE, related_name="segments", null=True
     )
 
-    # This defaults to 1 for newly created segments.
-    version = models.IntegerField(null=True)
+    version = models.IntegerField(default=1, null=True)
 
     version_of = models.ForeignKey(
         "self",
@@ -134,11 +133,6 @@ def get_skip_create_audit_log(self) -> bool:
 
         return False
 
-    @hook(BEFORE_CREATE, when="version_of", is_now=None)
-    def set_default_version_to_one_if_new_segment(self):  # type: ignore[no-untyped-def]
-        if self.version is None:
-            self.version = 1
-
     @hook(AFTER_CREATE, when="version_of", is_now=None)
     def set_version_of_to_self_if_none(self):  # type: ignore[no-untyped-def]
         """