10775

Committed 07 May 2026 03:11PM UTC coverage: 72.912% (+0.06%) from 72.848%

Build # 10775

Build Type

push

circleci

Committed by

web-flow

Commit Message

Merge pull request #370 from Freegle/feature/notification-chaseup-email

feat(batch): migrate notification_chaseup.php to Laravel

Coverage Stats

13809 of 20744 branches covered (66.57%)

Branch coverage included in aggregate %.

210 of 280 new or added lines in 6 files covered. (75.0%)

88 existing lines in 6 files now uncovered.

99064 of 134064 relevant lines covered (73.89%)

22.84 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

0.0

/iznik-batch/app/Console/Commands/Notification/ChaseUpNotificationsCommand.php

<?php

namespace App\Console\Commands\Notification;

use App\Services\NotificationChaseUpService;
use Illuminate\Console\Command;

/**
 * Send chaseup emails for unseen, unmailed site notifications (comments, loves, etc.).
 *
 * Migrated from V1 notification_chaseup.php → Notifications::sendEmails().
 * Runs every 5 minutes in production (see routes/console.php).
 */
class ChaseUpNotificationsCommand extends Command
{
    protected $signature = 'mail:notifications:chaseup
        {--user= : Only process this user ID}
        {--before=5 : Minimum age of notification in minutes before sending}
        {--since=24 : Maximum age of notification in hours}
        {--dry-run : Show what would be sent without sending}';

    protected $description = 'Send chaseup emails for unseen site notifications (comments, loves, etc.)';

    public function handle(NotificationChaseUpService $service): int
    {
        $userId        = $this->option('user') ? (int) $this->option('user') : null;
        $beforeMinutes = (int) $this->option('before');
        $sinceHours    = (int) $this->option('since');
        $dryRun        = (bool) $this->option('dry-run');

        if ($dryRun) {
            $this->info('Dry run — no emails will be sent.');
        }

        $count = $service->sendEmails(
            userId:        $userId,
            beforeMinutes: $beforeMinutes,
            sinceHours:    $sinceHours,
            dryRun:        $dryRun
        );

        $this->info("Sent {$count} notification chaseup email(s).");

        return Command::SUCCESS;
    }
}

1	<?php
2
3	namespace App\Console\Commands\Notification;
4
5	use App\Services\NotificationChaseUpService;
6	use Illuminate\Console\Command;
7
8	/**
9	* Send chaseup emails for unseen, unmailed site notifications (comments, loves, etc.).
10	*
11	* Migrated from V1 notification_chaseup.php → Notifications::sendEmails().
12	* Runs every 5 minutes in production (see routes/console.php).
13	*/
14	class ChaseUpNotificationsCommand extends Command
15	{
16	protected $signature = 'mail:notifications:chaseup
17	{--user= : Only process this user ID}
18	{--before=5 : Minimum age of notification in minutes before sending}
19	{--since=24 : Maximum age of notification in hours}
20	{--dry-run : Show what would be sent without sending}';
21
22	protected $description = 'Send chaseup emails for unseen site notifications (comments, loves, etc.)';
23
NEW 24	public function handle(NotificationChaseUpService $service): int	×
25	{
NEW 26	$userId = $this->option('user') ? (int) $this->option('user') : null;	×
NEW 27	$beforeMinutes = (int) $this->option('before');	×
NEW 28	$sinceHours = (int) $this->option('since');	×
NEW 29	$dryRun = (bool) $this->option('dry-run');	×
30
NEW 31	if ($dryRun) {	×
NEW 32	$this->info('Dry run — no emails will be sent.');	×
33	}
34
NEW 35	$count = $service->sendEmails(	×
NEW 36	userId: $userId,	×
NEW 37	beforeMinutes: $beforeMinutes,	×
NEW 38	sinceHours: $sinceHours,	×
NEW 39	dryRun: $dryRun	×
NEW 40	);	×
41
NEW 42	$this->info("Sent {$count} notification chaseup email(s).");	×
43
NEW 44	return Command::SUCCESS;	×
45	}
46	}

Freegle / Iznik / 10775

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous