I updated the Docusign link to the correct powerform. I also added a URL querystring param to the docusign powerform with the Member ID. that way, merging all the different datasets later will be much more straightforward.
I didn't test this manually since it requires external API calls. But the functional tests all pass and I look through those logs to check things were behaving as expected. The main break point would be if getDataValue('id') fails if for some reason 'id' doesn't exist. I tested that case too, and it'll just pass an empty string and nothing errors out.
after this PR, I believe we're ready to re-deploy. (short of the domain issues)
I updated the Docusign link to the correct powerform. I also added a URL querystring param to the docusign powerform with the Member ID. that way, merging all the different datasets later will be much more straightforward.
I didn't test this manually since it requires external API calls. But the functional tests all pass and I look through those logs to check things were behaving as expected. The main break point would be if getDataValue('id') fails if for some reason 'id' doesn't exist. I tested that case too, and it'll just pass an empty string and nothing errors out.
after this PR, I believe we're ready to re-deploy. (short of the domain issues)