berkayk / laravel-onesignal

OneSignal Push Notifications for Laravel
506 stars 176 forks source link

Bug in getNotifications methos #189

Open diogocoutinho opened 7 months ago

diogocoutinho commented 7 months ago

The getNotifications with a bug of write.

Error:

In the line 430 of OneSignalClient:

        if($offset) {
            $endpoint.="&offset=".$$offset;
        }

Here seems a double "$" and caused a error when provide a offset by user

Solution:

Just remove a "$"

        if($offset) {
            $endpoint.="&offset=".$offset;
        }

I will a PR to help.