thinktomorrow / chief

app/Notifications/InvitationMail.php

<?php

namespace Thinktomorrow\Chief\App\Notifications;

use Illuminate\Bus\Queueable;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Notifications\Messages\MailMessage;
use Illuminate\Notifications\Notification;
use Illuminate\Queue\SerializesModels;
use Thinktomorrow\Chief\Admin\Users\Invites\Invitation;

class InvitationMail extends Notification implements ShouldQueue
{
    use Queueable;
    use SerializesModels;

The trait Illuminate\Queue\SerializesModels requires some properties which are not provided by Thinktomorrow\Chief\App\Notifications\InvitationMail: $collectionClass, $relations, $class, $keyBy

    public $invitation;

    public function __construct(Invitation $invitation)
    {
        $this->invitation = $invitation;
    }

    /**
     * @return string[]
     *
     * @psalm-return array{0: string}
     */
    public function via($notifiable): array

The parameter $notifiable is not used and could be removed.

    {
        return ['mail'];
    }

    /**
     * Get the mail representation of the notification.
     *
     * @param  mixed $notifiable
     * @return \Illuminate\Notifications\Messages\MailMessage
     */
    public function toMail($notifiable)

The parameter $notifiable is not used and could be removed.

    {
        return (new MailMessage())
            ->subject('Uitnodiging tot Chief')
            ->from(chiefSetting('from_email', null, chiefSetting('contact_email')), chiefSetting('from_name', null, chiefSetting('contact_name')))
            ->view('chief::mails.invitation', [
                'invitee' => $this->invitation->invitee,
                'inviter' => $this->invitation->inviter,
                'accept_url' => $this->invitation->acceptUrl(),
                'deny_url' => $this->invitation->denyUrl(),
            ]);
    }

    /**
     * Get the array representation of the notification.
     *
     * @param  mixed $notifiable
     * @return array
     */
    public function toArray($notifiable)

The parameter $notifiable is not used and could be removed.

    {
        return [
            //
        ];
    }
}