[Vortex] FCM: Update response class for the new firebase v1 API

Review Request #1090 — Created Jan. 11, 2024 and submitted

Information
b.stoop
Lunr.Vortex
fcm/updateresponse
1089, 1102
1126
62cd815...
Reviewers
lunr 
Based on master


FCM: Update response class for the new firebase v1 API


The response changed as well, see the response json files in the diff. Referenced: https://firebase.google.com/docs/reference/fcm/rest/v1/FcmError. This probably needs some testing and tweaking once we have more responses from firebase as the documentation is not very clear or even inconsistent
Github actions run: https://github.com/brianstoop/lunr.vortex/actions/runs/8068348457
  • 0
  • 0
  • 2
  • 1
  • 3
Description From Last Updated
b.stoop
smillernl
smillernl 
  1. 
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    2. 

















  2. 
 
    
  
  
    
  
 

   
    
    
    



    
 
    
  
    
   
   
    
    
     
     
     
     
     
    
    Show all issues
   
  
    
 
    

    



    
    What does this have to do with FCM?
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    3. 




  3. 
 
    
  
    
   
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    4. 




  

 
















 



 

  

   




b.stoop




   

   

  

 



 















 





 

  

   




pprkut




   

   







  

 



 

  


   

   

    
 
pprkut


    
   


  

  

  



    




  1. 
 
    
  
    
   
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    2. 

















  2. 
 
    
  
  
    
  
 
    
  
    
   
    
    
     
      src/Lunr/Vortex/FCM/FCMResponse.php
      


       (Diff revision 3)


      
     
    
    
       
   
    
     
    

    
     
    

       
  
    
 
    

   
    
    
    



    
 
    
  
    
   
   
    
    
     
     
     
     
     
    
    Show all issues
   
  
    
 
    

    



    
    Similar to the payloads. If we need platform specific response handling, since we have platform specific payloads already, we can platform specific responses for fcm-android, fcm-apns and fcm-webpush too.
    

    There should still be a way to handle responses for mixed payload, so subclassing is probably not the solution to look for here, but something that splits up the code into separate components more.
    
  
    
 
    

 
    
  


    
 
    
 
      

  


    1. 
 
 
      
 
      
  
 
      
 
      
  
      Removed the apns response for this review as we don't have enough data/info for now
      
 
      

      2. 


 
    


    

 
    

    3. 




  3. 
 
    
  
    
   
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    4. 




  

 
















 



 

  

   




b.stoop




   

   

  

 



 















 





 

  

   




pprkut




   

   









  

 



 

  


   

   

    
 
pprkut


    
   


  

  

  



    




  1. 
 
    
  
    
   
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    2. 







  2. 
 
    
  
  
    
  
   
    
    
    



    
 
    
  
    
   
   
    
    
     
     
     
     
     
    
    Show all issues
   
  
    
 
    

    



    
    Needs a rebase, but then it should be fine
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    3. 













  3. 
 
    
  
    
   
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    4. 




  

 
















 



 

  

   




b.stoop




   

   

  

 



 















 



 

  

   




b.stoop




   

   

  

 



 

  


   

   

    
 Review request changed

    
   


  

  

  






 

  Status:
  

  Closed (submitted)

  
 




 
Change Summary:

 
Pushed to master (62cd815)





 


 










  

 









     

    

   

  

  

   
   Loading...