From 3981bee8f7eac381f73aff08063bb7949d8d6ecc Mon Sep 17 00:00:00 2001 From: Miguel Nogueira Date: Sun, 10 Aug 2025 11:03:29 +0100 Subject: [PATCH] fix: remove unnecessary notified flag updates in invitation processing this flag should only be used by post-processing jobs (e.g. invite lifecycle cleanup) Signed-off-by: Miguel Nogueira --- app/Http/Controllers/InvitationController.php | 2 -- 1 file changed, 2 deletions(-) diff --git a/app/Http/Controllers/InvitationController.php b/app/Http/Controllers/InvitationController.php index 39bf9c3..8f1085d 100644 --- a/app/Http/Controllers/InvitationController.php +++ b/app/Http/Controllers/InvitationController.php @@ -38,7 +38,6 @@ class InvitationController extends Controller $invitation->requestor_email = $request->input('email'); $invitation->requestor_ip_address = $request->ip(); $invitation->status = $guest ? 'pending' : 'approved'; - $invitation->notified = !$guest; // confirmation msg doesn't count $invitation->invitation_code = bin2hex(random_bytes(64)); $invitation->expiration = now()->addDays(2); @@ -78,7 +77,6 @@ class InvitationController extends Controller if ($invitation->expiration && now()->lessThanOrEqualTo($invitation->expiration) && in_array($invitation->status, $approvableStates)) { $invitation->status = 'approved'; - $invitation->notified = true; $invitation->save(); Mail::to($invitation->requestor_email)->send(new InviteApprovedMail($invitation));