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
pprkut
  1. 
      
  2. src/Lunr/Vortex/GCM/GCMBatchResponse.php (Diff revision 1)
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     

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

  3. src/Lunr/Vortex/GCM/GCMDispatcher.php (Diff revision 1)
     
     
     
     
     
     
     
     
     

    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)
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     

    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)
     
     

    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...