Add logging
This commit is contained in:
parent
8c08cbf912
commit
bb3f9b343c
2 changed files with 23 additions and 5 deletions
|
@ -2,8 +2,9 @@
|
||||||
|
|
||||||
namespace App\Console\Commands;
|
namespace App\Console\Commands;
|
||||||
|
|
||||||
use Illuminate\Console\Command;
|
|
||||||
use App\Mail\Digest;
|
use App\Mail\Digest;
|
||||||
|
use Illuminate\Console\Command;
|
||||||
|
use Illuminate\Support\Facades\Log;
|
||||||
use Illuminate\Support\Facades\Mail;
|
use Illuminate\Support\Facades\Mail;
|
||||||
|
|
||||||
class SendDigest extends Command
|
class SendDigest extends Command
|
||||||
|
@ -33,23 +34,25 @@ class SendDigest extends Command
|
||||||
$this->error('No schedule specified.');
|
$this->error('No schedule specified.');
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
// These are seperated because I may want to send multiple types
|
// These are seperated because I may want to send multiple types
|
||||||
// of digest in a single commend.
|
// of digest in a single commend.
|
||||||
if ($this->option('daily')) {
|
if ($this->option('daily')) {
|
||||||
foreach (config('app.daily_digest_recipients') as $recipient) {
|
foreach (config('app.daily_digest_recipients') as $recipient) {
|
||||||
|
Log::debug("Daily digest email sent to '{$recipient}'.");
|
||||||
Mail::to($recipient)->send(new Digest('daily', config('app.current_trip_id')));
|
Mail::to($recipient)->send(new Digest('daily', config('app.current_trip_id')));
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
if ($this->option('weekly')) {
|
if ($this->option('weekly')) {
|
||||||
foreach (config('app.weekly_digest_recipients') as $recipient) {
|
foreach (config('app.weekly_digest_recipients') as $recipient) {
|
||||||
|
Log::debug("Weekly digest email sent to '{$recipient}'.");
|
||||||
Mail::to($recipient)->send(new Digest('weekly', config('app.current_trip_id')));
|
Mail::to($recipient)->send(new Digest('weekly', config('app.current_trip_id')));
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
if ($this->option('monthly')) {
|
if ($this->option('monthly')) {
|
||||||
foreach (config('app.monthly_digest_recipients') as $recipient) {
|
foreach (config('app.monthly_digest_recipients') as $recipient) {
|
||||||
|
Log::debug("Monthly digest email sent to '{$recipient}'.");
|
||||||
Mail::to($recipient)->send(new Digest('monthly', config('app.current_trip_id')));
|
Mail::to($recipient)->send(new Digest('monthly', config('app.current_trip_id')));
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
@ -4,6 +4,7 @@ namespace App\Console;
|
||||||
|
|
||||||
use Illuminate\Console\Scheduling\Schedule;
|
use Illuminate\Console\Scheduling\Schedule;
|
||||||
use Illuminate\Foundation\Console\Kernel as ConsoleKernel;
|
use Illuminate\Foundation\Console\Kernel as ConsoleKernel;
|
||||||
|
use Illuminate\Support\Facades\Log;
|
||||||
|
|
||||||
class Kernel extends ConsoleKernel
|
class Kernel extends ConsoleKernel
|
||||||
{
|
{
|
||||||
|
@ -15,9 +16,23 @@ class Kernel extends ConsoleKernel
|
||||||
*/
|
*/
|
||||||
protected function schedule(Schedule $schedule)
|
protected function schedule(Schedule $schedule)
|
||||||
{
|
{
|
||||||
$schedule->command('digest:send --daily')->daily();
|
$schedule->command('digest:send --daily')
|
||||||
$schedule->command('digest:send --weekly')->weekly();
|
->daily()
|
||||||
$schedule->command('digest:send --monthly')->monthly();
|
->onFailure(function() {
|
||||||
|
Log::error("Daily email digest send failed");
|
||||||
|
});
|
||||||
|
|
||||||
|
$schedule->command('digest:send --weekly')
|
||||||
|
->weekly()
|
||||||
|
->onFailure(function() {
|
||||||
|
Log::error("Weekly email digest send failed");
|
||||||
|
});
|
||||||
|
|
||||||
|
$schedule->command('digest:send --monthly')
|
||||||
|
->monthly()
|
||||||
|
->onFailure(function() {
|
||||||
|
Log::error("Monthly email digest send failed");
|
||||||
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
|
Loading…
Reference in a new issue