Skip to content

Commit

Permalink
fix(notifications): ignore global webhook flag for dashboard notifica…
Browse files Browse the repository at this point in the history
…tions

remove unusued parameter
  • Loading branch information
peterbitfly committed Dec 9, 2024
1 parent f875e7d commit 2950978
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions backend/pkg/notification/queuing.go
Original file line number Diff line number Diff line change
Expand Up @@ -678,9 +678,8 @@ func QueueWebhookNotifications(notificationsByUserID types.NotificationsPerUserI
LEFT JOIN users_val_dashboards ON users_val_dashboards_groups.dashboard_id = users_val_dashboards.id
WHERE users_val_dashboards.user_id = ANY($1)
AND webhook_target IS NOT NULL
AND webhook_format IS NOT NULL
AND user_id NOT IN (SELECT user_id from users_notification_channels WHERE active = false and channel = $2);
`, pq.Array(userIds), types.WebhookNotificationChannel)
AND webhook_format IS NOT NULL;
`, pq.Array(userIds))
if err != nil {
return fmt.Errorf("error quering users_val_dashboards_groups, err: %w", err)
}
Expand Down

0 comments on commit 2950978

Please sign in to comment.