You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There is a method resetNotifier in the SendsPasswordResetEmails trait, but it is not called at all. Maybe we should remove it, or add a parameter in PasswordBroker::sendResetLink just like PasswordBroker::reset does.
From the PHPDoc, I guess this method would be used to return a notification like this:
/** * Get the Closure which is used to build the password reset notification. * * @return \Closure */protectedfunctionresetNotifier()
{
returnfunction ($token) {
returnnewResetPasswordNotification($token);
};
}
Am I right? Then what notification instance will be used if the notifiable object implemented CanResetPassword trait?
The text was updated successfully, but these errors were encountered:
You're right, this method is not used and should be deleted. Removed in #15446
To use a custom Notification you need to overwrite the sendPasswordResetNotification in your User model, this method is coming from the Illuminate\Auth\Passwords\CanResetPassword trait.
There is a method
resetNotifier
in theSendsPasswordResetEmails
trait, but it is not called at all. Maybe we should remove it, or add a parameter inPasswordBroker::sendResetLink
just likePasswordBroker::reset
does.From the PHPDoc, I guess this method would be used to return a notification like this:
Am I right? Then what notification instance will be used if the notifiable object implemented
CanResetPassword
trait?The text was updated successfully, but these errors were encountered: