Skip to content
This repository was archived by the owner on Jun 28, 2024. It is now read-only.

Commit 6d43a2c

Browse files
committed
Merge branch 'rasmuscnielsen-master'
2 parents 2efca8c + bb48fcd commit 6d43a2c

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

README.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,8 @@ Or you can add it directly in your composer.json file:
3838

3939
Register the provider directly in your app configuration file config/app.php `config/app.php`:
4040

41+
Laravel >= 5.5 provides package auto-discovery, thanks to rasmuscnielsen and luiztessadri who help to implement this feature in Laravel-FCM, the registration of the provider and the facades should not be necessary anymore.
42+
4143
```php
4244
'providers' => [
4345
// ...

src/Message/PayloadNotification.php

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -146,8 +146,10 @@ public function toArray()
146146
];
147147

148148
// remove null values
149-
$notification = array_filter($notification);
150-
149+
$notification = array_filter($notification, function($value) {
150+
return $value !== null;
151+
});
152+
151153
return $notification;
152154
}
153155
}

0 commit comments

Comments
 (0)