Use UniqueConstraint instead of unique_together for membership

orgs/migrations/0003_remove_org_unique_domain_name_when_not_deleted_and_more.py

@@ -0,0 +1,36 @@
+# Generated by Django 4.2.7 on 2024-07-23 11:45
+
+from django.db import migrations, models
+import django.db.models.deletion
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('app_users', '0019_alter_appusertransaction_reason'),
+        ('orgs', '0002_alter_org_unique_together_and_more'),
+    ]
+
+    operations = [
+        migrations.RemoveConstraint(
+            model_name='org',
+            name='unique_domain_name_when_not_deleted',
+        ),
+        migrations.AlterUniqueTogether(
+            name='orgmembership',
+            unique_together=set(),
+        ),
+        migrations.AlterField(
+            model_name='orginvitation',
+            name='status_changed_by',
+            field=models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.SET_NULL, related_name='received_invitations', to='app_users.appuser'),
+        ),
+        migrations.AddConstraint(
+            model_name='org',
+            constraint=models.UniqueConstraint(condition=models.Q(('deleted__isnull', True)), fields=('domain_name',), name='unique_domain_name_when_not_deleted', violation_error_message='This domain name is already in use by another team. Contact Gooey.AI Support <[email protected]> if you think this is a mistake.'),
+        ),
+        migrations.AddConstraint(
+            model_name='orgmembership',
+            constraint=models.UniqueConstraint(condition=models.Q(('deleted__isnull', True)), fields=('org', 'user'), name='unique_org_user'),
+        ),
+    ]

orgs/models.py

@@ -170,7 +170,13 @@ class OrgMembership(SafeDeleteModel):
     objects = SafeDeleteManager()
 
     class Meta:
-        unique_together = ("org", "user", "deleted")
+        constraints = [
+            models.UniqueConstraint(
+                fields=["org", "user"],
+                condition=Q(deleted__isnull=True),
+                name="unique_org_user",
+            )
+        ]
 
     def __str__(self):
         return f"{self.get_role_display()} - {self.user} ({self.org})"