Laravel - should send notification method

Question

I have this configuration where the user can set which notification it wants to receive, I don't want to do an If on every method that I call notify.

I'd like to know if is there a method inside my notification class that I can do this validation, or how could I do that.

I thought about a solution, but it seens durty, I could validate inside via and just return an empty array if the user setted to not receive

And I also find out a method inside Illuminate\Notifications\NotificationSender called shouldSendNotification but I don't know how I could overwrite it, or even if it is using this class, cause it seens to be only for queue

OBS: Laravel 7

Dwight · Accepted Answer · 2021-10-18 10:02:45Z

3

via would previously have been the best place for this, but Laravel now supports shouldSend on the notification for exactly this behaviour.

/**
 * Determine if the notification should be sent.
 *
 * @param  mixed  $notifiable
 * @param  string  $channel
 * @return bool|null
 */
public function shouldSend($notifiable, $channel)
{
    if ($user->isUnsubscribed()) {
        return false;
    }
}

answered Oct 18, 2021 at 10:02

Dwight

12.5k6 gold badges53 silver badges67 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

MCP · Accepted Answer · 2020-08-07 21:13:00Z

0

I guess the easiest way is to define a wrapper method in your User model and use it instead of notify like this:

public function _notify($notification)
{
    if ($this->notify_condition) {
        $this->notify($notification);
    }
}

now you use _notify instead of notify

answered Aug 7, 2020 at 21:13

MCP

1171 silver badge11 bronze badges

Collectives™ on Stack Overflow

Laravel - should send notification method

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related