fix: handle no subscriber on bulk notifications

goto · Jun 4, 2024 · b9029d7 · b9029d7
1 parent 3a318e3
commit b9029d7
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/core/notification/dispatch_bulk_notification_service.go b/core/notification/dispatch_bulk_notification_service.go
@@ -102,6 +102,11 @@ func (s *DispatchBulkNotificationService) Dispatch(ctx context.Context, ns []Not
 		return nil, err
 	}
 
+	if len(metaMessages) == 0 {
+		s.deps.Logger.Info("no meta messages to process")
+		return nil, nil
+	}
+
 	if err := s.deps.LogService.LogNotifications(ctx, notificationLogs...); err != nil {
 		return nil, fmt.Errorf("failed logging notifications: %w", err)
 	}