$emailAddress must be a string. Got: [email protected]

[isaacdarcilla]

[isaacdarcilla]

Default should be $notifiable->email->value instead of $notifiable->email.

[aozisik]

Hi @isaacdarcilla,

The readme was out of date. Actually we use the routeNotificationFor('mail') method. This is the correct interface method to use for notifiables, and it's expected to return the email address as a string. You might want to check your implementation of this method in the User class (or your notifiable model).

https://laravel.com/docs/10.x/notifications#customizing-the-recipient

In case you don't want to touch your model code, you can always add a custom "->to" directive and pass the email address that way instead. If you provide at least one "to" address to the SendGridMessage instance, the library will use that instead.

I'm closing this issue for now, but feel free to re-open if necessary.

[isaacdarcilla]

Hi @isaacdarcilla,

The readme was out of date. Actually we use the routeNotificationFor('mail') method. This is the correct interface method to use for notifiables, and it's expected to return the email address as a string. You might want to check your implementation of this method in the User class (or your notifiable model).

https://laravel.com/docs/10.x/notifications#customizing-the-recipient

In case you don't want to touch your model code, you can always add a custom "->to" directive and pass the email address that way instead. If you provide at least one "to" address to the SendGridMessage instance, the library will use that instead.

I'm closing this issue for now, but feel free to re-open if necessary.

Thank you for the details.

[aozisik]

Hi @isaacdarcilla,

This might not affect the specific issue that you encountered, but I just realised the routeNotificationForMail can also return an array (['email_address' => 'name']) as well.

I opened a PR for this, and will release this soon.