Skip to content

Commit 588840a

Browse files
fix: Resolve group by id max length issue (#36668)
1 parent 5bc0581 commit 588840a

2 files changed

Lines changed: 19 additions & 1 deletion

File tree

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
# Generated by Django 4.2.20 on 2025-05-06 13:17
2+
3+
from django.db import migrations, models
4+
5+
6+
class Migration(migrations.Migration):
7+
8+
dependencies = [
9+
('notifications', '0006_notification_group_by_id'),
10+
]
11+
12+
operations = [
13+
migrations.AlterField(
14+
model_name='notification',
15+
name='group_by_id',
16+
field=models.CharField(db_index=True, default='', max_length=255),
17+
),
18+
]

openedx/core/djangoapps/notifications/models.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ class Notification(TimeStampedModel):
111111
email = models.BooleanField(default=False, null=False, blank=False)
112112
last_read = models.DateTimeField(null=True, blank=True)
113113
last_seen = models.DateTimeField(null=True, blank=True)
114-
group_by_id = models.CharField(max_length=42, db_index=True, null=False, default="")
114+
group_by_id = models.CharField(max_length=255, db_index=True, null=False, default="")
115115

116116
def __str__(self):
117117
return f'{self.user.username} - {self.course_id} - {self.app_name} - {self.notification_type}'

0 commit comments

Comments
 (0)