Fix notification role handling to support both enum and string role values

src/app/services/notification_creator.py

@@ -47,7 +47,7 @@ Usage:
 import logging
 from sqlalchemy.orm import Session
 from uuid import UUID
-from typing import Optional, List, Dict, Any
+from typing import Optional, List, Dict, Any, Union
 from datetime import datetime
 
 from app.models.notification import Notification, NotificationChannel, NotificationStatus
@@ -228,7 +228,7 @@ class NotificationCreator:
         source_type: str,
         source_id: Optional[UUID],
         notification_type: str,
-        roles: Optional[List[AppRole]] = None,
+        roles: Optional[List[Union[str, AppRole]]] = None,
         channel: str = "in_app",
         metadata: Optional[Dict[str, Any]] = None,
         exclude_user_ids: Optional[List[UUID]] = None
@@ -279,7 +279,16 @@ class NotificationCreator:
 
             # Filter by roles if specified
             if roles:
-                role_values = [role.value for role in roles]
+                # Handle both string roles and AppRole enum objects
+                role_values = []
+                for role in roles:
+                    if isinstance(role, str):
+                        role_values.append(role)
+                    elif hasattr(role, 'value'):
+                        role_values.append(role.value)
+                    else:
+                        role_values.append(str(role))
+                
                 query = query.filter(User.role.in_(role_values))
 
             # Exclude specific users if specified