Page MenuHomePhabricator

Comb Amazon out of our Mustache
Closed, ResolvedPublic1 Estimated Story Points

Description

Undo the mistakes made in the rush to get Amazon in Mustache. It's better to have amazon.js create a couple of divs with jQuery than it is to have atrocious is_amazon variables. Also go back to using x_required context variables rather than deeply nested method_metadata.validation.x.

This will make it easier to merge the Amazon branch on top of the form changes made for Worldpay.

Event Timeline

Ejegg raised the priority of this task from to Needs Triage.
Ejegg updated the task description. (Show Details)
Ejegg added a subscriber: Ejegg.
Ejegg set Security to None.
Ejegg edited a custom field.

Change 238661 had a related patch set uploaded (by Ejegg):
Bring Amazon branch in line with WP templating

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

Change 238661 merged by jenkins-bot:
Bring Amazon branch back to normal Mustache templates

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