[Vortex] JPush payload generation

Review Request #698 — Created July 15, 2020 and submitted

smillernl
Lunr
feature/vortex/jpush_payload
HKG-1262
696
699
lunr
Vortex: JPush payload generation


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

    How is that relevant for the payload?

    1. The structure of the payload will be like this, so we need to know what platforms we support

      --
        platform: ['ios', 'android']
        audience: []
        notification:
          android:
            stuff: things
          ios:
            stuff: things
      
  3. 
      
pprkut
  1. 
      
  2. src/Lunr/Vortex/JPush/JPushPayload.php (Diff revision 1)
     
     
    The issue has been resolved. Show all issues

    The existing payload classes to a json_encode() mostly for legacy reasons. New payloads shouldn't do this anymore.

  3. 
      
smillernl
smillernl
pprkut
  1. Ship It!
  2. 
      
smillernl
Review request changed

Status: Closed (submitted)

Loading...