Page MenuHomePhabricator

reimplement adyen iframe in a variant
Closed, ResolvedPublic2 Estimated Story Points

Description

As an advancement manager, I would like the adyen iframe as a variant (not default) so that I can assess how usable it is compared to the hosted payment page.

DoD:

  • re-implement this as a variant
  • ensure it is not going out as default before and after deploying
  • Inform fr-online of the new variant name

Event Timeline

DStrine set the point value for this task to 2.Sep 1 2020, 8:13 PM
DStrine moved this task from Sprint +1 to Current Sprint on the Fundraising-Backlog board.

@DStrine It looks like the previous iframe code (that we switched off due to the cookie bug last month) was detecting whether the donor was using either an Apple tablet or phone OR was wanting to use IDEAL via Adyen and if either of those cases were true we'd redirect the donors to the Adyen's payment pages and not show the iframe. Do we wanna keep that behaviour for the new iframe variant also?

To be clear, if we kept the original behaviour the iframe variant would be ignored even if passed if the donor was using an Apple phone, tablet or they wanted to donate using IDEAL via Adyen

Ok, it looks like we redirect for apple devices as the iframe wasn't working in safari. I guess we could test whether this is still the case and if so, keep the original redirect behaviour for these devices.

Change 626219 had a related patch set uploaded (by Jgleeson; owner: Jgleeson):
[mediawiki/extensions/DonationInterface@master] WIP: Added adyenIframe variant.

https://gerrit.wikimedia.org/r/626219

When testing I was able to get the continue button to show up with the skins but I am getting an error message on continue

Invalid Request
Unfortunately we were unable to process this request.
Perhaps your browser does not accept cookies or the previous page was a saved or cached copy?

Error:
(Note: these error messages are not visible on the live platform)

The redirect flow is working fine and I'm not seeing anything weird in the syslog.

So I tried this locally and it works for me. I wonder if they fix the bug since?

Here's my test

adyen-iframe-test (829×1 px, 2 MB)

Looks like I hadn't rebased over master when testing previously in the screen capture above which meant I was still seeing the three-step Adyen checkout process. I can confirm after rebasing the patch over latest master I now see the expected two-step process and it completes successfully on my end.

I did @mepps the variant in the iframe loads for me but it gives me an error when I try and complete a payment.

My skin codes and browser version match the ones @jgleeson is using and non iframe payments process fine so not sure what is going on with my local setup.

@Cstone could you screenshot and post what you see if you get a chance? thanks!

invalidrequest.png (686×508 px, 53 KB)

Okay this is happening when I open the iframe in an incognito window

Change 626219 merged by jenkins-bot:
[mediawiki/extensions/DonationInterface@master] Added adyenIframe variant.

https://gerrit.wikimedia.org/r/626219