orchestral / notifier

Notifier Component for Orchestra Platform

Notifier Component for Orchestra Platform

Notifier Component add a simplify approach to notifier the application user using mail (or other notification service) that is managed using Orchestra\Notifier\NotifierManager.

Version Compatibility

Laravel	Notifier
5.5.x	3.5.x
5.6.x	3.6.x
5.7.x	3.7.x
5.8.x	3.8.x
6.x	4.x
7.x	5.x
8.x	6.x

Installation

To install through composer, run the following command from terminal:

composer require "orchestra/notifier"

Configuration

Add following service providers in config/app.php.

'providers' => [

    // ...

    Orchestra\Notifier\NotifierServiceProvider::class,

],

notifier's Issues

Add mandrill and mailgun API transport.

Run Travis-CI on nightly PHP 7 build for 2.1 and above.

README and some basic doc.

[Insight] Object parameters should be type hinted - in src/Notifier/LaravelNotifier.php, line 40

in src/Notifier/LaravelNotifier.php, line 40

The parameter mail, which is an object, should be typehinted.

        $view    = $message->view;
        $data    = $message->data;
        $subject = $message->subject;

        // Send the email directly using Illuminate\Contracts\Mail\Mailer interface.
        $this->mailer->send($view, $data, function ($mail) use ($user, $subject, $callback) {
            // Set the recipient detail.
            $mail->to($user->getRecipientEmail(), $user->getRecipientName());

            // Only append the subject if it was provided.
            ! empty($subject) && $mail->subject($subject);

---

Posted from SensioLabsInsight

[Insight] Unused use statement should be avoided

in src/Orchestra/Notifier/LaravelNotifier.php, line 5

The class Illuminate\Auth\Reminders\RemindableInterface is declared but never used. You should remove the use statement.

<?php namespace Orchestra\Notifier;

use Closure;
use Illuminate\Mail\Mailer as IlluminateMailer;
use Illuminate\Auth\Reminders\RemindableInterface;

class LaravelNotifier implements NotifierInterface
{
    /**
     * Mailer instance.

Posted from SensioLabsInsight

[Insight] Object parameters should be type hinted - in src/Notifier/Mailer.php, line 147

in src/Notifier/Mailer.php, line 147

The parameter job, which is an object, should be typehinted.

     *
     * @param  \Illuminate\Queue\Jobs\Job  $job
     * @param  array  $data
     * @return void
     */
    public function handleQueuedMessage($job, $data)
    {
        $this->send($data['view'], $data['data'], $this->getQueuedCallable($data));

        $job->delete();
    }

---

Posted from SensioLabsInsight

Release support for Laravel 6

[Insight] Unused use statement should be avoided

in src/Orchestra/Notifier/OrchestraNotifier.php, line 5

The class Illuminate\Auth\Reminders\RemindableInterface is declared but never used. You should remove the use statement.

<?php namespace Orchestra\Notifier;

use Closure;
use Orchestra\Memory\Abstractable\Container;
use Illuminate\Auth\Reminders\RemindableInterface;
use Illuminate\Support\SerializableClosure;

class OrchestraNotifier extends Container implements NotifierInterface
{
    /**

Posted from SensioLabsInsight

Allow to generate inline css for email template.

[Insight] Object parameters should be type hinted - in src/Notifier/OrchestraNotifier.php, line 50

in src/Notifier/OrchestraNotifier.php, line 50

The parameter message, which is an object, should be typehinted.

        // exception.
        $callback = ($callback instanceof Closure ? new SerializableClosure($callback) : $callback);

        // Send the notification using push which would allow Orchestra
        // Platform to choose either to use queue or send.
        $receipt = $this->mailer->push($view, $data, function ($message) use ($user, $subject, $callback) {
            // Set the recipient detail.
            $message->to($user->getRecipientEmail(), $user->getRecipientName());

            // Only append the subject if it was provided.
            ! empty($subject) && $message->subject($subject);

---

Posted from SensioLabsInsight

Update code to match latest changes to Mail::send() return value to null for all transport.

Add support to SES Transport.