From 546256b22cca128021280a25242882e9f352993e Mon Sep 17 00:00:00 2001
From: Andras Bacsai <5845193+andrasbacsai@users.noreply.github.com>
Date: Fri, 12 Dec 2025 11:12:19 +0100
Subject: [PATCH] Fix: Allow test emails to be sent to any email address
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Test emails should work with any recipient email address for verification purposes, not just team members. Added an isTestNotification flag to both Test notification classes and modified EmailChannel to skip team membership validation for test notifications.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
---
 app/Notifications/Channels/EmailChannel.php   | 35 +++++++++++--------
 app/Notifications/Test.php                    |  2 ++
 .../TransactionalEmails/Test.php              |  2 ++
 3 files changed, 24 insertions(+), 15 deletions(-)

diff --git a/app/Notifications/Channels/EmailChannel.php b/app/Notifications/Channels/EmailChannel.php
index 234bc37ad..abd115550 100644
--- a/app/Notifications/Channels/EmailChannel.php
+++ b/app/Notifications/Channels/EmailChannel.php
@@ -43,21 +43,26 @@ public function send(SendsEmail $notifiable, Notification $notification): void
                 throw new Exception('No email recipients found');
             }
 
-            foreach ($recipients as $recipient) {
-                // Check if the recipient is part of the team
-                if (! $members->contains('email', $recipient)) {
-                    $emailSettings = $notifiable->emailNotificationSettings;
-                    data_set($emailSettings, 'smtp_password', '********');
-                    data_set($emailSettings, 'resend_api_key', '********');
-                    send_internal_notification(sprintf(
-                        "Recipient is not part of the team: %s\nTeam: %s\nNotification: %s\nNotifiable: %s\nEmail Settings:\n%s",
-                        $recipient,
-                        $team,
-                        get_class($notification),
-                        get_class($notifiable),
-                        json_encode($emailSettings, JSON_PRETTY_PRINT | JSON_UNESCAPED_SLASHES)
-                    ));
-                    throw new Exception('Recipient is not part of the team');
+            // Skip team membership validation for test notifications
+            $isTestNotification = data_get($notification, 'isTestNotification', false);
+
+            if (! $isTestNotification) {
+                foreach ($recipients as $recipient) {
+                    // Check if the recipient is part of the team
+                    if (! $members->contains('email', $recipient)) {
+                        $emailSettings = $notifiable->emailNotificationSettings;
+                        data_set($emailSettings, 'smtp_password', '********');
+                        data_set($emailSettings, 'resend_api_key', '********');
+                        send_internal_notification(sprintf(
+                            "Recipient is not part of the team: %s\nTeam: %s\nNotification: %s\nNotifiable: %s\nEmail Settings:\n%s",
+                            $recipient,
+                            $team,
+                            get_class($notification),
+                            get_class($notifiable),
+                            json_encode($emailSettings, JSON_PRETTY_PRINT | JSON_UNESCAPED_SLASHES)
+                        ));
+                        throw new Exception('Recipient is not part of the team');
+                    }
                 }
             }
 
diff --git a/app/Notifications/Test.php b/app/Notifications/Test.php
index 60bc8a0ee..bbed22777 100644
--- a/app/Notifications/Test.php
+++ b/app/Notifications/Test.php
@@ -23,6 +23,8 @@ class Test extends Notification implements ShouldQueue
 
     public $tries = 5;
 
+    public bool $isTestNotification = true;
+
     public function __construct(public ?string $emails = null, public ?string $channel = null, public ?bool $ping = false)
     {
         $this->onQueue('high');
diff --git a/app/Notifications/TransactionalEmails/Test.php b/app/Notifications/TransactionalEmails/Test.php
index 3add70db2..2f7d70bbf 100644
--- a/app/Notifications/TransactionalEmails/Test.php
+++ b/app/Notifications/TransactionalEmails/Test.php
@@ -8,6 +8,8 @@
 
 class Test extends CustomEmailNotification
 {
+    public bool $isTestNotification = true;
+
     public function __construct(public string $emails, public bool $isTransactionalEmail = true)
     {
         $this->onQueue('high');