-
-
-
src/Lunr/Vortex/APNS/ApnsPHP/APNSDispatcher.php (Diff revision 1) Can you add a use declaration for it?
-
src/Lunr/Vortex/APNS/ApnsPHP/APNSDispatcher.php (Diff revision 1) no root namespace indicator needed
-
-
src/Lunr/Vortex/APNS/ApnsPHP/APNSDispatcher.php (Diff revision 1) We should be able to say APNSPayload here instead of object, and APNSResponse instead of PushNotificationResponseInterface
-
src/Lunr/Vortex/Email/EmailDispatcher.php (Diff revision 1) We should be able to say EmailPayload here instead of object, and EmailResponse instead of PushNotificationResponseInterface
-
-
src/Lunr/Vortex/FCM/FCMDispatcher.php (Diff revision 1) We should be able to say FCMPayload here instead of object, and FCMResponse instead of PushNotificationResponseInterface
-
-
-
-
src/Lunr/Vortex/JPush/JPushDispatcher.php (Diff revision 1) We should be able to say JPushPayload here instead of object, and JPushResponse instead of PushNotificationResponseInterface
-
-
src/Lunr/Vortex/MPNS/MPNSDispatcher.php (Diff revision 1) We should be able to say MPNSPayload here instead of object, and MPNSResponse instead of PushNotificationResponseInterface
-
-
-
-
-
src/Lunr/Vortex/PAP/PAPDispatcher.php (Diff revision 1) We should be able to say PAPPayload here instead of object, and PAPResponse instead of PushNotificationResponseInterface
-
-
-
src/Lunr/Vortex/WNS/WNSDispatcher.php (Diff revision 1) We should be able to say WNSPayload here instead of object, and WNSResponse instead of PushNotificationResponseInterface
-
[Vortex] General: Add types
Review Request #742 — Created March 10, 2021 and discarded
Information | |
---|---|
smillernl | |
Lunr.Vortex | |
feature/general/types | |
765 | |
Reviewers | |
lunr | |
General: Add types
- 1
- 0
- 21
- 2
- 24
Description | From | Last Updated |
---|---|---|
Sorry, I'm getting lost in this review. Can we split it up into one review per platform? | pprkut |