Vortex: send GCM push notification by batch

Review Request #448 — Created May 26, 2016 and submitted

tardypad
Lunr
push_notification_dispatcher_gcm_batch
lunr
Vortex: send GCM push notification by batch

unit tests

  • 0
  • 0
  • 4
  • 0
  • 4
Description From Last Updated
There are no open issues
pprkut
  1. 
      
  2. src/Lunr/Vortex/GCM/GCMBatchResponse.php (Diff revision 1)
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
    The issue has been resolved. Show all issues

    I'd rather have a default statement then having it set before the switch

  3. src/Lunr/Vortex/GCM/GCMDispatcher.php (Diff revision 1)
     
     
     
     
     
     
     
     
     
    The issue has been resolved. Show all issues

    pass by reference?

    I suppose it would make sense here as we'd potentially have a lot of endpoints.

  4. 
      
tardypad
pprkut
  1. 
      
  2. src/Lunr/Vortex/GCM/GCMDispatcher.php (Diff revisions 1 - 2)
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
    The issue has been resolved. Show all issues

    I see you made $this->push_batch() receive the parameter by reference, which is fine, and actually something I missed ;)

    What I actually meant before was the foreach

  3. src/Lunr/Vortex/GCM/GCMResponse.php (Diff revisions 1 - 2)
     
     
    The issue has been resolved. Show all issues

    We generally don't do this in Lunr. Type hinting provides contracts, and Lunr mostly provides contract free functionality.

  4. 
      
tardypad
pprkut
  1. Ship It!
  2. 
      
tardypad
Review request changed

Status: Closed (submitted)

Change Summary:

Merged into master

Loading...