Andy Gaskell

I think we could improve the documentation. Some parts of https://php-fcm.readthedocs.io/ are unclear or out of date. Just jotting this down as an issue so I remember to come back...

FCM: Migrate from legacy HTTP to HTTP v1

8

Just really writing this as a note, and/or a place for a discussion about it. So, at the moment we use the FCM API referred to as "HTTP", this is...

This is an odd one, but in FCM, if you add more than 5 topics, the message doesn't send. This is really unhelpful in Firebase. It's mentioned in the docs......

If you send a data only push, and empty message is sent

1

There is an issue in Push/Notification.php whereby if you want to send a data only message, so, you leave the title, body etc fields blank, it'll still send a non-data...

Think we just need 2 more tests to increase automated test coverage to 100%. It's at 98.9% at the moment on https://coveralls.io/github/EdwinHoksberg/php-fcm Add a test that passes data param into...

Removing the -n param

1

Looks like -n is removed from the latest uglifyjs, gives the error... error: unknown option `-n'

Tests not running as documented

2

In the Readme at... https://github.com/EdwinHoksberg/php-fcm/blob/master/README.md ...it describes running PHPUnit tests with... `composer install` ...and... `composer test` But this returns... `Command "test" is not defined.`

No Readme file in earlier steps

1

Hi Dimitris Hope you're well, it's been ages since I last saw you, in Köln (I think), it'd be great to have a beer together again. I was going to...