make (name, infra type) unique

Author: dogboat

dojo/db_migrations/0269_cicd_infrastructure.py

@@ -64,12 +64,13 @@ class Migration(migrations.Migration):
             fields=[
                 ("id", models.AutoField(auto_created=True, primary_key=True, serialize=False, verbose_name="ID")),
                 ("name", models.CharField(max_length=200)),
-                ("description", models.CharField(blank=True, max_length=2000, null=True)),
-                ("url", models.URLField(blank=True, help_text="Public URL of the tool (e.g., https://jenkins.company.com)", max_length=2000, null=True)),
+                ("description", models.CharField(blank=True, default="", max_length=2000)),
+                ("url", models.URLField(blank=True, default="", help_text="Public URL of the tool (e.g., https://jenkins.company.com)", max_length=2000)),
                 ("infrastructure_type", models.CharField(choices=[("build_server", "Build Server"), ("scm_server", "SCM Server"), ("orchestration", "Orchestration Engine")], max_length=30)),
             ],
             options={
                 "ordering": ["name"],
+                "unique_together": {("name", "infrastructure_type")},
             },
         ),
         # Step 2: Add new FK fields to Engagement (before removing old ones)

dojo/models.py

@@ -1345,12 +1345,13 @@ class CICDInfrastructure(models.Model):
     )
 
     name = models.CharField(max_length=200)
-    description = models.CharField(max_length=2000, null=True, blank=True)
-    url = models.URLField(max_length=2000, null=True, blank=True, help_text=_("Public URL of the tool (e.g., https://jenkins.company.com)"))
+    description = models.CharField(max_length=2000, blank=True, default="")
+    url = models.URLField(max_length=2000, blank=True, default="", help_text=_("Public URL of the tool (e.g., https://jenkins.company.com)"))
     infrastructure_type = models.CharField(max_length=30, choices=INFRASTRUCTURE_TYPE_CHOICES)
 
     class Meta:
         ordering = ["name"]
+        unique_together = [("name", "infrastructure_type")]
 
     def __str__(self):
         return self.name