Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Braintree - Multiple Merchant Account IDs for a multiple currency #235

Open
wants to merge 3 commits into
base: 2-4-stable
Choose a base branch
from

Conversation

tomaswitek
Copy link

Fix for this Issue: #236

Multiple currency setup

https://articles.braintreepayments.com/get-started/currencies#multi-currency-setups

Braintree setup

Add multiple Merchant Account IDs for each currency.
For every second currency use this pattern: myMerchantId-currency:

braintree

You should be able to do it with API calls in US.
In Europa you have to contact Braintree support team.

Spree setup

Choose the same currencies in Spree config.
It is important that the default currency matches the one in Braintree config.
spree-config

Specify the Braintree default Merchant ID in Spree config.
spree-payment

Single currency setup

https://articles.braintreepayments.com/get-started/currencies#single-currency-setups
Everything stays the same as before.

@tomaswitek tomaswitek changed the title Multiple merchant account ids for a multiple currency Braintree - Multiple merchant account ids for a multiple currency Sep 5, 2015
@tomaswitek tomaswitek changed the title Braintree - Multiple merchant account ids for a multiple currency Braintree - Multiple Merchant Account IDs for a multiple currency Sep 5, 2015
@JDutil
Copy link
Member

JDutil commented Sep 5, 2015

Please write specs for this.

@tomaswitek
Copy link
Author

Sure 😏

@tomaswitek
Copy link
Author

Ready for review

@kushniryb
Copy link
Contributor

@tomaswitek
Is it still valid?
Would you mind rebasing it against master & making sure that everything works like expected?
Sorry it took us so long to review it 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants