use routeNotificationFor to retrieve mail, if to's are not set

Author: aozisik

src/SendGridChannel.php

@@ -33,8 +33,22 @@ public function send($notifiable, Notification $notification)
             throw new Exception('You must implement toSendGrid in the notification class for SendGrid channel.');
         }
 
+        /**
+         * @var SendGridMessage
+         */
         $message = $notification->toSendGrid($notifiable);
 
+        if (empty($message->from)) {
+            $message->from(
+                config('mail.from.address'),
+                config('mail.from.name')
+            );
+        }
+
+        if (empty($message->tos)) {
+            $message->to($notifiable->routeNotificationFor('mail'));
+        }
+
         if (! ($message instanceof SendGridMessage)) {
             throw new Exception('toSendGrid must return an instance of SendGridMessage.');
         }

src/SendGridMessage.php

@@ -69,7 +69,7 @@ public function from($email, $name)
      * @param  array  $data
      * @return $this
      */
-    public function to($email, $name, $data = [])
+    public function to($email, $name = null, $data = [])
     {
         $this->tos = array_merge($this->tos, [new To($email, $name, $data)]);
 
@@ -88,14 +88,6 @@ public function payload($payload)
      */
     public function build(): Mail
     {
-        if (! $this->from) {
-            // If from is not defined, use the values set in Laravel config
-            $this->from(
-                config('mail.from.address'),
-                config('mail.from.name')
-            );
-        }
-
         $email = new Mail(
             $this->from,
             $this->tos