Page MenuHomePhabricator

GatewayAdapter::getCurrencies should return currencies supported for the donor's country
Closed, ResolvedPublic2 Estimated Story Points

Description

When a country is selected, getCurrencies should return only the currencies supported by that gateway in that country. It also doesn't need to be a static function.

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.

I like it. There's some potential for generalization, too, if we have some way of querying methods / gateways / currencies, given any known variables (method, country).

DStrine edited a custom field.

We could potentially get this info from the DonationInterfaceFormSettings.

Change 271569 had a related patch set uploaded (by Ejegg):
WIP filter currencies by country

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

Change 271689 had a related patch set uploaded (by Ejegg):
Don't show single-value currency dropdown

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

Change 271703 had a related patch set uploaded (by Ejegg):
Allow a different fallback currency for each country

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

Change 271569 merged by jenkins-bot:
Filter currencies by country

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

I would love to catch up on the design for this, there are a lot of really useful-seeming patches, but I don't quite get how the pieces fit together. Building for the AstroPay case might be a little bit over-specific. @Ejegg, Tomorrow would be a good time to chat actually because I can play a very convincing person-on-the-street who knows nothing about what's going on ;-)

Change 271703 merged by jenkins-bot:
Allow a different fallback currency for each country

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

Change 271689 merged by jenkins-bot:
Don't show single-value currency dropdown

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