From: Magnus Hagander <magnus@hagander.net>
Date: Fri, 9 Jan 2026 10:04:15 +0000 (+0100)
Subject: Make manager a required field on teams
X-Git-Url: http://git.postgresql.org/gitweb/download.html?a=commitdiff_plain;p=hamn.git

Make manager a required field on teams

This was left as NULLable when created to make the migration path
simpler, but it's now set on all teams and there are parts of the code
that relies on it being set, so enforce it in the database.
---

diff --git a/hamnadmin/hamnadmin/register/migrations/0008_require_team_manager.py b/hamnadmin/hamnadmin/register/migrations/0008_require_team_manager.py
new file mode 100644
index 0000000..61a41e6
--- /dev/null
+++ b/hamnadmin/hamnadmin/register/migrations/0008_require_team_manager.py
@@ -0,0 +1,21 @@
+# Generated by Django 4.2.11 on 2026-01-09 10:03
+
+from django.conf import settings
+from django.db import migrations, models
+import django.db.models.deletion
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        migrations.swappable_dependency(settings.AUTH_USER_MODEL),
+        ('register', '0007_moderatornotes'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='team',
+            name='manager',
+            field=models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, to=settings.AUTH_USER_MODEL),
+        ),
+    ]
diff --git a/hamnadmin/hamnadmin/register/models.py b/hamnadmin/hamnadmin/register/models.py
index 66c99ea..f4e3b86 100644
--- a/hamnadmin/hamnadmin/register/models.py
+++ b/hamnadmin/hamnadmin/register/models.py
@@ -8,7 +8,7 @@ from hamnadmin.util.shortlink import urlvalmap
 class Team(models.Model):
     teamurl = models.CharField(max_length=255, blank=False)
     name = models.CharField(max_length=255, blank=False)
-    manager = models.ForeignKey(User, null=True, blank=True, on_delete=models.CASCADE)
+    manager = models.ForeignKey(User, null=False, blank=False, on_delete=models.CASCADE)
 
     def __str__(self):
         return "%s (%s)" % (self.name, self.teamurl)