fix: modify the notifications serializer and WS url

thirdeye-dev · Jul 18, 2023 · 21c1978 · 21c1978 · vercel · Jul 18, 2023
1 parent e60efee
commit 21c1978
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 7 deletions.
diff --git a/backend/api_app/monitoring/routing.py b/backend/api_app/monitoring/routing.py
@@ -4,6 +4,6 @@
 
 websocket_urlpatterns = [
     path(
-        "ws/notifications/<str:owner_organization_id>/", NotificationsConsumer.as_asgi()
+        "ws/notifications/<str:owner_organization_id>", NotificationsConsumer.as_asgi()
     ),
 ]
diff --git a/backend/api_app/monitoring/serializers.py b/backend/api_app/monitoring/serializers.py
@@ -153,12 +153,6 @@ def validate_configuration(yaml_data):
     return serializer.validated_data
 
 
-class NotificationAPISerializer(rfs.ModelSerializer):
-    class Meta:
-        model = Notification
-        exclude = ["task_id"]
-
-
 class AlertsAPISerializer(rfs.ModelSerializer):
     class CustomYAMLField(rfs.Field):
         @classmethod
@@ -301,3 +295,13 @@ def send_webhook(self, alert_data):
             trigger_transaction_hash=self.transaction.hash,
         )
         notification.save()
+
+class NotificationAPISerializer(rfs.ModelSerializer):
+    alert_name = rfs.CharField(source="alert.name")
+    alert_description = rfs.CharField(source="alert.description")
+    contract_name = rfs.CharField(source="alert.smart_contract.name")
+
+    class Meta:
+        model = Notification
+        exclude = ["task_id"]
+