Problem

Remote post should be more flexible with fully integrated response confirmation

Solution

Refactor remote post to handle all submission states. Allow response to be inserted into submission data, emails, and confirmation pages via tokens

Steps

  • Improve test coverage
  • Write webform example remote post
  • Restructure remote post to support all states
  • Add webform-remote-post token support. Only supports current post.
  • Add preprocess callback to WebformHandler
  • Write confirmation code test
  • Write confirmation page test
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jrockowitz created an issue. See original summary.

  • jrockowitz committed 7aaf7e4 on 2898424-remote-post
    Issue #2898424: Improve Remote Post. Fix tests and create an example.
    
jrockowitz’s picture

Status: Active » Needs review
FileSize
20.16 KB

  • jrockowitz committed b458a2b on 2898424-remote-post
    Issue #2898424: Improve Remote Post. Add all states and custom header...

  • jrockowitz committed b22b51c on 2898424-remote-post
    Issue #2898424: Improve Remote Post. Add [webform_handler] token support...

  • jrockowitz committed 589293b on 2898424-remote-post
    Issue #2898424: Improve Remote Post. Update example
    
jrockowitz’s picture

Status: Needs review » Needs work

The last submitted patch, 7: improve_remote_post-2898424-7.patch, failed testing. View results

  • jrockowitz committed 81a63f8 on 2898424-remote-post
    Issue #2898424: Improve Remote Post. Fix broken tests.
    
jrockowitz’s picture

Status: Needs work » Needs review
FileSize
74.56 KB

  • jrockowitz committed c6ac264 on 2898424-remote-post
    Issue #2898424: Improve Remote Post. Fix example
    

  • jrockowitz committed ffc8a02 on 8.x-5.x
    Issue #2898424 by jrockowitz: Improve Remote Post
    
jrockowitz’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.