fix(Notifications): Ensure markAllRead api call actually flags all of…

… the logged in users notifications as being read.
PurdueRCAC · Apr 11, 2024 · 6dc4257 · 6dc4257
1 parent 415500c
commit 6dc4257
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/app/Modules/Users/Http/Controllers/Api/NotificationsController.php b/app/Modules/Users/Http/Controllers/Api/NotificationsController.php
@@ -11,6 +11,7 @@
 use App\Modules\Users\Http\Resources\NotificationResourceCollection;
 use App\Modules\Users\Http\Resources\NotificationResource;
 use App\Modules\Users\Models\User;
+use Carbon\Carbon;
 
 /**
  * User Notifications
@@ -302,6 +303,12 @@ public function markAllRead()
 			->whereNull('read_at')
 			->get();
 
+		foreach($rows as $row)
+		{
+			$row->markAsRead();
+			$row->save();
+		}
+
 		return new NotificationResourceCollection($rows);
 	}