feat(event): add crisis severity level history if only ifrc_severity_level is updated

Author: AdityaKhatri

api/admin.py

@@ -6,6 +6,7 @@
 from django.contrib.auth.admin import UserAdmin
 from django.contrib.auth.models import User
 from django.contrib.gis import admin as geoadmin
+from django.core.exceptions import ValidationError
 from django.http import HttpResponse, HttpResponseRedirect
 from django.urls import reverse
 from django.utils.html import format_html, format_html_join
@@ -272,6 +273,28 @@ def response_change(self, request, obj):
             return HttpResponseRedirect(".")
         return super().response_change(request, obj)
 
+    def save_model(self, request, obj, form, change):
+        if change:
+            # Fetch original object from DB to compare
+            original = models.Event.objects.get(pk=obj.pk)
+
+            severity_changed = original.ifrc_severity_level != obj.ifrc_severity_level
+            update_date_changed = original.ifrc_severity_level_update_date != obj.ifrc_severity_level_update_date
+
+            if severity_changed and not update_date_changed:
+                messages.error(request, "You must update the 'IFRC Severity Level Update Date' when changing the severity level.")
+                raise ValidationError("Cannot change severity level without updating the update date.")
+
+            if severity_changed and update_date_changed:
+                models.EventSeverityLevelHistory.objects.create(
+                    event=obj,
+                    ifrc_severity_level=obj.ifrc_severity_level,
+                    ifrc_severity_level_update_date=obj.ifrc_severity_level_update_date,
+                    created_by=request.user,
+                )
+
+        super().save_model(request, obj, form, change)
+
     def field_reports(self, instance):
         if getattr(instance, "field_reports").exists():
             return format_html_join(
@@ -1014,6 +1037,16 @@ def has_add_permission(cls, request, obj=None):
         return request.user.is_superuser
 
 
+@admin.register(models.EventSeverityLevelHistory)
+class EventSeverityLevelHistoryAdmin(admin.ModelAdmin):
+    list_select_related = True
+    list_display = ["event", "ifrc_severity_level", "created_by", "created_at"]
+    autocomplete_fields = (
+        "event",
+        "created_by",
+    )
+
+
 @admin.register(models.Export)
 class ExportTokenAdmin(admin.ModelAdmin):
     pass

api/drf_views.py

@@ -42,6 +42,7 @@
     DistrictFilter,
     DistrictRMDFilter,
     EventFilter,
+    EventSeverityLevelHistoryFilter,
     EventSnippetFilter,
     FieldReportFilter,
     GoHistoricalFilter,
@@ -83,6 +84,7 @@
     District,
     Event,
     EventFeaturedDocument,
+    EventSeverityLevelHistory,
     Export,
     ExternalPartner,
     FieldReport,
@@ -123,6 +125,7 @@
     DisasterTypeSerializer,
     DistrictSerializer,
     DistrictSerializerRMD,
+    EventSeverityLevelHistorySerializer,
     ExportSerializer,
     ExternalPartnerSerializer,
     FieldReportGeneratedTitleSerializer,
@@ -212,6 +215,12 @@ def get_queryset(self):
         )
 
 
+class EventSeverityLevelHistoryViewSet(viewsets.ReadOnlyModelViewSet):
+    queryset = EventSeverityLevelHistory.objects.select_related("event", "created_by").order_by("-created_at")
+    serializer_class = EventSeverityLevelHistorySerializer
+    filter_class = EventSeverityLevelHistoryFilter
+
+
 class DeployedERUFilter(rest_filters.FilterSet):
     eru_type = rest_filters.NumberFilter(field_name="eru__type", lookup_expr="exact")
 

api/filter_set.py

@@ -16,6 +16,7 @@
     CountrySupportingPartner,
     District,
     Event,
+    EventSeverityLevelHistory,
     FieldReport,
     Region,
     RegionKeyFigure,
@@ -40,6 +41,19 @@ class Meta:
         fields = ()
 
 
+class EventSeverityLevelHistoryFilter(filters.FilterSet):
+    # NOTE: Adding this fixed N + 1 for some reason
+    # Getting this issue: NO SCROLL CURSOR WITH HOLD FOR SELECT
+    event = filters.NumberFilter(field_name="event", lookup_expr="exact")
+
+    class Meta:
+        model = EventSeverityLevelHistory
+        fields = {
+            "id": ("exact", "in"),
+            "event": ("exact", "in"),
+        }
+
+
 class CountryFilter(filters.FilterSet):
     region = filters.NumberFilter(field_name="region", lookup_expr="exact")
     record_type = filters.NumberFilter(field_name="record_type", lookup_expr="exact")

api/migrations/0220_event_ifrc_severity_level_update_date_and_more.py

@@ -0,0 +1,49 @@
+# Generated by Django 4.2.19 on 2025-08-01 08:59
+
+import django.db.models.deletion
+from django.conf import settings
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        migrations.swappable_dependency(settings.AUTH_USER_MODEL),
+        ("api", "0219_alter_appealfilter_value"),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name="event",
+            name="ifrc_severity_level_update_date",
+            field=models.DateTimeField(blank=True, null=True, verbose_name="ifrc severity level update date"),
+        ),
+        migrations.CreateModel(
+            name="EventSeverityLevelHistory",
+            fields=[
+                ("id", models.AutoField(auto_created=True, primary_key=True, serialize=False, verbose_name="ID")),
+                (
+                    "ifrc_severity_level",
+                    models.IntegerField(
+                        choices=[(0, "Yellow"), (1, "Orange"), (2, "Red")], default=0, verbose_name="IFRC Severity level"
+                    ),
+                ),
+                (
+                    "ifrc_severity_level_update_date",
+                    models.DateTimeField(blank=True, null=True, verbose_name="ifrc severity level update date"),
+                ),
+                ("created_at", models.DateTimeField(auto_now_add=True, verbose_name="created at")),
+                (
+                    "created_by",
+                    models.ForeignKey(
+                        null=True,
+                        on_delete=django.db.models.deletion.SET_NULL,
+                        related_name="created_by_ifrc_severity_level",
+                        to=settings.AUTH_USER_MODEL,
+                        verbose_name="created by",
+                    ),
+                ),
+                ("event", models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, to="api.event", verbose_name="event")),
+            ],
+        ),
+    ]

api/models.py

@@ -737,6 +737,8 @@ def snippet_image_path(instance, filename):
     return "emergencies/%s/%s" % (instance.id, filename)
 
 
+# NOTE: If ever in future we need to create an api to update the event table
+# we also need to make sure to add appropriate signal to create ifrc severity level event history
 @reversion.register()
 class Event(models.Model):
     """A disaster, which could cover multiple countries"""
@@ -789,6 +791,9 @@ class Event(models.Model):
     num_displaced = models.IntegerField(verbose_name=_("number of displaced"), null=True, blank=True)
 
     ifrc_severity_level = models.IntegerField(choices=AlertLevel.choices, default=0, verbose_name=_("IFRC Severity level"))
+    ifrc_severity_level_update_date = models.DateTimeField(
+        verbose_name=_("ifrc severity level update date"), null=True, blank=True
+    )
     glide = models.CharField(verbose_name=_("glide"), max_length=18, blank=True)
 
     created_at = models.DateTimeField(verbose_name=_("created at"), auto_now_add=True)
@@ -888,6 +893,29 @@ def __str__(self):
         return self.name
 
 
+class EventSeverityLevelHistory(models.Model):
+    # FIXME: Enum value shouldn't contain 0 or 1
+    ifrc_severity_level = models.IntegerField(choices=AlertLevel.choices, default=0, verbose_name=_("IFRC Severity level"))
+    ifrc_severity_level_update_date = models.DateTimeField(
+        verbose_name=_("ifrc severity level update date"),
+        null=True,
+        blank=True,
+    )
+    created_at = models.DateTimeField(verbose_name=_("created at"), auto_now_add=True)
+    event = models.ForeignKey(
+        Event,
+        verbose_name=_("event"),
+        on_delete=models.CASCADE,
+    )
+    created_by = models.ForeignKey(
+        settings.AUTH_USER_MODEL,
+        verbose_name=_("created by"),
+        on_delete=models.SET_NULL,
+        null=True,
+        related_name="created_by_ifrc_severity_level",
+    )
+
+
 @reversion.register()
 class EventFeaturedDocument(models.Model):
     event = models.ForeignKey(

api/serializers.py

@@ -49,6 +49,7 @@
     EventContact,
     EventFeaturedDocument,
     EventLink,
+    EventSeverityLevelHistory,
     Export,
     ExternalPartner,
     FieldReport,
@@ -119,6 +120,12 @@ class Meta:
         )
 
 
+class EventSeverityLevelHistorySerializer(ModelSerializer):
+    class Meta:
+        model = EventSeverityLevelHistory
+        fields = "__all__"
+
+
 class RegionGeoSerializer(ModelSerializer):
     bbox = serializers.SerializerMethodField()
 
@@ -1032,6 +1039,7 @@ class Meta:
             "num_affected",
             "ifrc_severity_level",
             "ifrc_severity_level_display",
+            "ifrc_severity_level_update_date",
             "glide",
             "disaster_start_date",
             "created_at",
@@ -1080,6 +1088,7 @@ class Meta:
             "num_affected",
             "ifrc_severity_level",
             "ifrc_severity_level_display",
+            "ifrc_severity_level_update_date",
             "glide",
             "disaster_start_date",
             "created_at",
@@ -1127,6 +1136,7 @@ class Meta:
             "num_affected",
             "ifrc_severity_level",
             "ifrc_severity_level_display",
+            "ifrc_severity_level_update_date",
             "glide",
             "disaster_start_date",
             "created_at",
@@ -1171,6 +1181,7 @@ class Meta:
             "num_affected",
             "ifrc_severity_level",
             "ifrc_severity_level_display",
+            "ifrc_severity_level_update_date",
             "glide",
             "disaster_start_date",
             "created_at",
@@ -1341,6 +1352,7 @@ class Meta:
             "tab_one_title",
             "ifrc_severity_level",
             "ifrc_severity_level_display",
+            "ifrc_severity_level_update_date",
             "parent_event",
             "glide",
             "featured_documents",

api/test_models.py

@@ -1,13 +1,17 @@
 from unittest.mock import patch
 
+from django.contrib.admin.sites import AdminSite
 from django.contrib.auth.models import User
-from django.test import TestCase
+from django.test import RequestFactory, TestCase
+from django.utils import timezone
 from rest_framework.test import APITestCase
 
 import api.models as models
+from api.admin import EventAdmin
 from api.factories import country as countryFactory
 from api.factories import event as eventFactory
 from api.factories import field_report as fieldReportFactory
+from api.factories.region import RegionFactory
 from main.mock import erp_request_side_effect_mock
 
 
@@ -128,3 +132,49 @@ def setUp(self):
     def test_profile_create(self):
         obj = models.Profile.objects.get(user__username="test1")
         self.assertEqual(obj.department, "testdepartment")
+
+
+class EventSeverityLevelHistoryTest(TestCase):
+
+    fixtures = ["DisasterTypes"]
+
+    def setUp(self):
+        self.user = User.objects.create(username="username", first_name="pat", last_name="smith", password="password")
+        self.region = RegionFactory()
+        self.country = countryFactory.CountryFactory()
+        self.dtype = models.DisasterType.objects.get(pk=1)
+
+        self.event = eventFactory.EventFactory.create(
+            dtype=self.dtype,
+            ifrc_severity_level=models.AlertLevel.YELLOW,
+        )
+        self.event.regions.set([self.region])
+        self.event.countries.set([self.country])
+        self.event._current_user = self.user
+        self.admin = EventAdmin(models.Event, AdminSite())
+        self.factory = RequestFactory()
+
+    def test_ifrc_severity_level_history_created_from_admin(self):
+        # Simulate admin request
+        request = self.factory.post("/")
+        request.user = self.user
+
+        # Update the instance
+        self.event.ifrc_severity_level = models.AlertLevel.RED
+        self.event.ifrc_severity_level_update_date = timezone.now()
+
+        # Trigger admin save_model (where your logic lives)
+        self.admin.save_model(request, self.event, None, change=True)
+
+        history = models.EventSeverityLevelHistory.objects.filter(event=self.event)
+        self.assertEqual(history.count(), 1)
+        self.assertEqual(history[0].ifrc_severity_level, models.AlertLevel.RED)
+        self.assertEqual(history[0].created_by, self.user)
+
+    def test_no_history_created_if_severity_level_not_changed(self):
+        # Save the event with the current severity level (no change)
+        self.event.ifrc_severity_level = models.AlertLevel.YELLOW  # same as initial value
+        self.event.save()
+
+        history = models.EventSeverityLevelHistory.objects.filter(event=self.event)
+        self.assertEqual(history.count(), 0)

main/urls.py

@@ -98,6 +98,9 @@
 router.register(r"eru-readiness", deployment_views.ERUReadinessViewSet, basename="eru_readiness")
 router.register(r"eru-readiness-type", deployment_views.ERUReadinessTypeViewset, basename="eru_readiness_type")
 router.register(r"event", api_views.EventViewset, basename="event")
+router.register(
+    r"event-severity-level-history", api_views.EventSeverityLevelHistoryViewSet, basename="event_severity_level_history"
+)
 router.register(r"go-historical", api_views.GoHistoricalViewSet, basename="go_historical")
 router.register(r"featured_event_deployments", api_views.EventDeploymentsViewset, basename="featured_event_deployments")
 router.register(r"field-report", api_views.FieldReportViewset, basename="field_report")