CouldNotSendNotification thrown even when notification was sent successfully

[tusharvikky]

Hello,

"Notification was not sent. You should specify device token(s), topic(s) or group(s) for sending notification."

However, laravel-fcm.log logs
[2017-05-13 20:37:29] Laravel-FCM.INFO: notification send to 1 devices success: 1 failures: 0 number of modified token : 0 [] [] and the push notification was successfully delivered.

[enniel]

show your code in routeNotificationForFCM

[tusharvikky]

@enniel It has a comma separated list of tokens. Any info if you have to sent notification to n devices of a single user?

[tusharvikky]

Any update?

[enniel]

Show your code in routeNotificationForFCM and your notification file. I need to test your code.

[tusharvikky]

    public function routeNotificationForFcm()
    {
        $tokens = implode(",", PushToken::where('user_id', $this->id)->pluck('token')->toArray());
        return $tokens;
    } 

Notification File:

<?php

namespace App\Notifications;

use App\Models\PushToken;
use Illuminate\Bus\Queueable;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Notifications\Messages\MailMessage;
use Illuminate\Notifications\Notification;
use Illuminate\Support\Facades\Config;
use Illuminate\Support\Facades\Log;
use NotificationChannels\FCM\FCMMessage;


class ClassroomCreated extends Notification implements ShouldQueue
{
    use Queueable;

    private $classroom;
    private $tokens;

    /**
     * Create a new notification instance.
     *
     * @return void
     */
    public function __construct($classroom)
    {
        $this->classroom = $classroom;
    }

    /**
     * Get the notification's delivery channels.
     *
     * @param  mixed  $notifiable
     * @return array
     */
    public function via($notifiable)
    {
        return ['fcm'];
    }

    public function toFCM($notifiable)
    {
        try {
            // dd($this->classroom);
            $message = Config::get('gb.notification_messages.classroom');
            $rand_message = $message[array_rand($message, 1)];
            return (new FCMMessage())
                // ->to( $this->tokens )
                ->notification([
                    'title' => $rand_message,
                    'body' => $this->classroom->name,
                    'clickAction' => $this->classroom->id
                ]);            
        } catch (\Exception $e) {
                // dd($e);
            Log::error($e);
        }

    }

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

[enniel]

You tryed array of tokens?

    public function routeNotificationForFCM()
    {
        return PushToken::where('user_id', $this->id)->pluck('token')->toArray();
    } 

[tusharvikky]

Sadly, that didn't help either.

[2017-05-17 19:29:36] local.ERROR: exception 'NotificationChannels\FCM\Exceptions\CouldNotSendNotification' with message 'Notification was not sent. You should specify device token(s), topic(s) or group(s) for sending notification.' in /<project>/vendor/enniel/laravel-fcm-notification-channel/src/Exceptions/CouldNotSendNotification.php:14

[enniel]

You used routeNotificationForFCM or routeNotificationForFcm method?

[tusharvikky]

Tried w/ both. Didn't work. Still the same.

[enniel]

You can show output from dd(PushToken::where('user_id', $this->id)->pluck('token')->toArray()) ?

[tusharvikky]

[enniel]

I'm tried send notification and all works:

// notifiable model
<?php

namespace App;

use Illuminate\Notifications\Notifiable;
use Illuminate\Foundation\Auth\User as Authenticatable;

class User extends Authenticatable
{
    use Notifiable;

    /**
     * The attributes that are mass assignable.
     *
     * @var array
     */
    protected $fillable = [
        'name', 'email', 'password',
    ];

    /**
     * The attributes that should be hidden for arrays.
     *
     * @var array
     */
    protected $hidden = [
        'password', 'remember_token',
    ];


    public function routeNotificationForFCM() {
      return ['fYuuanS_YK4:APA91bEsFBKF3cUXotoxvOv2S9kyss9V3ONQZUuDq8cBvm57AuXdJn6SB3sY5JMpe0lLJUPk4D7c8bWTtPLRkwykJNtnazNMmBbo8eoU8mctiTZXf5XvN_lA_h7Mf9o2AUg9gMBgfmKG'];
    }
}

// notification
<?php

namespace App\Notifications;

use NotificationChannels\FCM\FCMMessage;
use Illuminate\Notifications\Notification;

class ExampleNotification extends Notification
{
    public function via($notifiable)
    {
        return ['fcm'];
    }

    public function toFCM($notifiable)
    {
        return (new FCMMessage())
            ->notification([
                'title' => 'Notification title',
                'body' => 'Notification body',
            ]);
    }
}

// send notififications
Notification::send(User::all(), new ExampleNotification());

[tusharvikky]

This is so weird. Only two things different is queue able and that try catch block.

[enniel]

Maybe

    public function routeNotificationForFCM() {
      return PushToken::where('user_id', $this->id)->whereNotNull('token')->pluck('token')->toArray();
    }

[enniel]

You can show your notifiable model code?