From b2153a7f5d6ef8d9998c643744fdbe957137eaed Mon Sep 17 00:00:00 2001 From: Wout Gevaert Date: Thu, 26 Mar 2026 12:54:04 +0100 Subject: [PATCH] Set notification pushed before creating echo event. Because for some reason this is not done in our production, but people are getting emails. HOTFIX, we should make a better fix later, e.g. have both 'pending' and 'pushed' status --- src/NotificationRunner.php | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/NotificationRunner.php b/src/NotificationRunner.php index 40b4634..af69bd2 100644 --- a/src/NotificationRunner.php +++ b/src/NotificationRunner.php @@ -68,13 +68,13 @@ public function run(): void { $data = $notification['data'] ?? []; $data['type'] = $notifier->getName(); - // Create and send the notification - $this->echoEventCreator->create( $data ); - if ( $notificationId !== null ) { // Make sure we don't push the notification twice by storing that we have sent it $this->pushedNotificationBucket->setPushed( $notificationId ); } + + // Create and send the notification + $this->echoEventCreator->create( $data ); } }