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

Add NGN and ZAR fiat currencies #2214

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Zero-1729
Copy link

This PR adds support for the Nigerian Naira (NGN) and South African Rand (ZAR).

CoinGecko supports both currencies, see https://api.coingecko.com/api/v3/simple/supported_vs_currencies.

Note: currency included in CoinGecko supported fiat list: https://api.coingecko.com/api/v3/simple/supported_vs_currencies

- added NGN fiat code
- added NGN to account fiat list
- added NGN to fiat currencies list
- added NGN to balance and fee tests
- Updated CHANGELOG to reflect addition.
Note: currency included in CoinGecko supported fiat list: https://api.coingecko.com/api/v3/simple/supported_vs_currencies

- added ZAR fiat code
- added ZAR to account fiat list
- added ZAR to fiat currencies list
- added ZAR to balance and fee tests
- Updated CHANGELOG to reflect addition.
@benma
Copy link
Contributor

benma commented Aug 8, 2023

Thanks for your contribution! Unfortunately we cannot merge this for the time being as we first need to make optimizations to our currency conversion backend, which is feeling the load of all the currency pairs currently supported!

It's not clear when we will get to this either, as it's quite low prio at the moment.

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.

2 participants