Skip to content

Commit

Permalink
Merge pull request #4120 from dwightwatson/patch-1
Browse files Browse the repository at this point in the history
Use notification argument in routing examples
  • Loading branch information
taylorotwell authored Feb 25, 2018
2 parents 1c5f9f1 + 8761a6e commit 32c5eec
Showing 1 changed file with 6 additions and 3 deletions.
9 changes: 6 additions & 3 deletions notifications.md
Original file line number Diff line number Diff line change
Expand Up @@ -254,9 +254,10 @@ When sending notifications via the `mail` channel, the notification system will
/**
* Route notifications for the mail channel.
*
* @param \Illuminate\Notifications\Notification $notification
* @return string
*/
public function routeNotificationForMail()
public function routeNotificationForMail($notification)
{
return $this->email_address;
}
Expand Down Expand Up @@ -622,9 +623,10 @@ When sending notifications via the `nexmo` channel, the notification system will
/**
* Route notifications for the Nexmo channel.
*
* @param \Illuminate\Notifications\Notification $notification
* @return string
*/
public function routeNotificationForNexmo()
public function routeNotificationForNexmo($notification)
{
return $this->phone;
}
Expand Down Expand Up @@ -798,9 +800,10 @@ To route Slack notifications to the proper location, define a `routeNotification
/**
* Route notifications for the Slack channel.
*
* @param \Illuminate\Notifications\Notification $notification
* @return string
*/
public function routeNotificationForSlack()
public function routeNotificationForSlack($notification)
{
return $this->slack_webhook_url;
}
Expand Down

0 comments on commit 32c5eec

Please sign in to comment.