Merge pull request #175 from pneumaticapp/backend/admin/43697__error_when_deleting_user

43697 backend [ Command panel ] Error 500 when deleting an account or user

Author: pneumojoseph

backend/src/accounts/migrations/0141_notification_text_default.py

@@ -0,0 +1,21 @@
+# Generated by Django 2.2 on 2026-03-17 03:38
+
+from django.db import migrations, models
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('accounts', '0140_alter_user_photo_max_length'),
+    ]
+
+    operations = [
+        migrations.RunSQL(
+            "UPDATE accounts_notification SET text = '' WHERE text IS NULL;",
+            migrations.RunSQL.noop
+        ),
+        migrations.AlterField(
+            model_name='notification',
+            name='text',
+            field=models.TextField(default=''),
+        ),
+    ]

backend/src/accounts/models.py

@@ -568,7 +568,7 @@ class Notification(
         on_delete=models.CASCADE,
         related_name='notifications',
     )
-    text = models.TextField(null=True)
+    text = models.TextField(default='')
     type = models.CharField(max_length=24, choices=NotificationType.CHOICES)
     status = models.CharField(
         max_length=10,

backend/src/notifications/tests/test_tasks/test_send_urgent.py

@@ -82,7 +82,7 @@ def test_send_urgent_notification__call_services__ok(mocker):
         author_id=user.id,
         account_id=account.id,
         type=NotificationType.URGENT,
-        text=None,
+        text='',
     )
     websocket_service_init_mock.assert_called_once_with(
         logo_lg=account.logo_lg,
@@ -161,7 +161,7 @@ def test_send_urgent_notification__call_services_with_group__ok(mocker):
         author_id=user.id,
         account_id=account.id,
         type=NotificationType.URGENT,
-        text=None,
+        text='',
     )
     websocket_service_init_mock.assert_called_once_with(
         logo_lg=account.logo_lg,
@@ -230,7 +230,7 @@ def test_send_not_urgent_notification__call_services__ok(mocker):
         author_id=user.id,
         account_id=account.id,
         type=NotificationType.NOT_URGENT,
-        text=None,
+        text='',
     )
     websocket_service_init_mock.assert_called_once_with(
         logo_lg=account.logo_lg,
@@ -296,7 +296,7 @@ def test_send_urgent_notification_completed_performer__skip(mocker):
         author_id=user.id,
         account_id=account.id,
         type=NotificationType.URGENT,
-        text=None,
+        text='',
     ).exists()
 
     websocket_urgent_mock.assert_not_called()
@@ -352,7 +352,7 @@ def test_send_urgent_notification_deleted_performer__skip(mocker):
         author_id=user.id,
         account_id=account.id,
         type=NotificationType.URGENT,
-        text=None,
+        text='',
     ).exists()
 
     websocket_urgent_mock.assert_not_called()
@@ -406,7 +406,7 @@ def test_send_urgent_notification_guest_performer__skip(mocker):
         author_id=user.id,
         account_id=account.id,
         type=NotificationType.URGENT,
-        text=None,
+        text='',
     ).exists()
 
     websocket_urgent_mock.assert_not_called()
@@ -459,7 +459,7 @@ def test_send_urgent_notification__another_task__skip(mocker):
         author_id=user.id,
         account_id=account.id,
         type=NotificationType.URGENT,
-        text=None,
+        text='',
     )
 
     link = f'{settings.FRONTEND_URL}/tasks/{task_2.id}'