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