From 786262db3b57fc70957dad94abb23ce8689f3115 Mon Sep 17 00:00:00 2001 From: Dane Everitt Date: Thu, 13 Apr 2017 20:03:03 -0400 Subject: [PATCH 1/2] Queue confirmation emails. Implements Laravel's queue abilities into the email notification job. Should not affect existing installations that are not using queues as the environment file defaults to `sync`. --- app/Notifications/ConfirmEmail.php | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/app/Notifications/ConfirmEmail.php b/app/Notifications/ConfirmEmail.php index 64d9bb9ac..3a4916748 100644 --- a/app/Notifications/ConfirmEmail.php +++ b/app/Notifications/ConfirmEmail.php @@ -3,11 +3,14 @@ namespace BookStack\Notifications; use Illuminate\Notifications\Notification; +use Illuminate\Contracts\Queue\ShouldQueue; use Illuminate\Notifications\Messages\MailMessage; -class ConfirmEmail extends Notification +class ConfirmEmail extends Notification implements ShouldQueue { + use Queueable; + public $token; /** From dfaf20dd83a838be36a0492c37e055a46b449e77 Mon Sep 17 00:00:00 2001 From: Dane Everitt Date: Thu, 13 Apr 2017 20:09:38 -0400 Subject: [PATCH 2/2] Actually include the Queueable namespace... --- app/Notifications/ConfirmEmail.php | 1 + 1 file changed, 1 insertion(+) diff --git a/app/Notifications/ConfirmEmail.php b/app/Notifications/ConfirmEmail.php index 3a4916748..27ac89c32 100644 --- a/app/Notifications/ConfirmEmail.php +++ b/app/Notifications/ConfirmEmail.php @@ -2,6 +2,7 @@ namespace BookStack\Notifications; +use Illuminate\Bus\Queueable; use Illuminate\Notifications\Notification; use Illuminate\Contracts\Queue\ShouldQueue; use Illuminate\Notifications\Messages\MailMessage;