Vortex: add ApnsPHP dispatcher
Review Request #446 — Created May 26, 2016 and submitted — Latest diff uploaded
Information | |
---|---|
tardypad | |
Lunr | |
apns_php_dispatcher | |
|
|
Reviewers | |
lunr | |
The new ApnsPHP is pretty annoying to work with inside our notification structure in the end...
see http://reviews.lunr.nl/r/447/ as well for the response classIt's not straightforward to define the status of the push for each endpoint:
- the library only returns an error structure (in a weird format...) for the endpoints with a failure
- some invalid endpoints are detected early on before the push and we don't get them as part of this report (because they are detected in a different Message class)
- we can then assume that the notification was delivered successfully to the endpoints left
unit tests
Diff Revision 5
This is not the most recent revision of the diff. The latest diff is revision 6. See what's changed.
src/Lunr/Vortex/APNS/ApnsPHP/APNSDispatcher.php |
---|
src/Lunr/Vortex/APNS/ApnsPHP/Tests/APNSDispatcherBaseTest.php |
---|
src/Lunr/Vortex/APNS/ApnsPHP/Tests/APNSDispatcherPushTest.php |
---|
src/Lunr/Vortex/APNS/ApnsPHP/Tests/APNSDispatcherSetTest.php |
---|
src/Lunr/Vortex/APNS/ApnsPHP/Tests/APNSDispatcherTest.php |
---|
tests/phpunit.xml |
---|