Skip to content

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

Open
tomaswitek wants to merge 3 commits into
spree-contrib:2-4-stablefrom
panter:features/braintree-multi-currency
Open

Braintree - Multiple Merchant Account IDs for a multiple currency#235
tomaswitek wants to merge 3 commits into
spree-contrib:2-4-stablefrom
panter:features/braintree-multi-currency

Conversation

@tomaswitek
Copy link
Copy Markdown

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
Copy Markdown
Member

JDutil commented Sep 5, 2015

Please write specs for this.

@tomaswitek
Copy link
Copy Markdown
Author

Sure 😏

@tomaswitek
Copy link
Copy Markdown
Author

Ready for review

@kushniryb
Copy link
Copy Markdown
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