Make a separate variant in case we need to switch over due to iframe bugs. Iframe variant should not be affected.
Description
Details
Event Timeline
Read https://phabricator.wikimedia.org/T176913 for background on the same task for adyen
Change 475221 had a related patch set uploaded (by Ejegg; owner: Ejegg):
[mediawiki/extensions/DonationInterface@master] Simple override to force redirect
Change 476766 had a related patch set uploaded (by XenoRyet; owner: XenoRyet):
[mediawiki/extensions/DonationInterface@master] Ingenico Full Redirect
That change does a full redirect when isIframe returns false, but the resulting page looks kind of silly as it's styled for the iFrame. To be donor-ready we probably need to have something better looking there, but this does function as-is.
I'll look into making it possible to select different ingenico-side variants depending on whether or not we're in an iFrame. That kind of change isn't something we'd normally do in a code freeze, but I'll try to have something ready in case we need it for some kind of Unbreak Now situation.
Created an Ingenico-side variant. It's still pretty simple, but it has the logo and is recognizable as part of our flow. It's on the Configuration Center if anyone wants to check it out.
Change 479009 had a related patch set uploaded (by Ejegg; owner: Ejegg):
[mediawiki/extensions/DonationInterface@master] Full redirect for Ingenico
Change 475221 merged by jenkins-bot:
[mediawiki/extensions/DonationInterface@master] Simple override to force redirect
Change 479009 merged by jenkins-bot:
[mediawiki/extensions/DonationInterface@master] Full redirect for Ingenico
As per the conversation in standup today, while this is rough around the edges, it does serve the intended purpose of an emergency backup option. Moving to done, and maybe we can have a new ticket to polish it up more later.
Change 476766 abandoned by XenoRyet:
Ingenico Full Redirect
Reason:
Competing patch was a better solution