-
Notifications
You must be signed in to change notification settings - Fork 4
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
Option to show fiat value #52
Comments
Shd a wallet contain mulitple fiat? If the wallet is currently in USD, will the amount in BTC be displayed? Side note: I think switching fiat is a major case (?) So it shd be in main wallet screen instead of settings screen |
I think only conversion to one currency at the same time might occur. If the currency is switched, then, if the main wallet is set to show fiat instead of NAV, it should show the chosen currency. So it would show https://api.coingecko.com/api/v3/simple/price?ids=nav-coin&vs_currencies=USD,EUR So switching the currency would not need additional fetches but just setting the I would not complicate the main screen with more elements. How do you think it'd be integrated there instead of in settings? |
Sounds like a fun feature, I will work on it these two days |
I think it won't complicate the main screen a lot as long as we use hooks to handle it, which if we use settings page, we wud still use hooks |
Conscious decision after discussing with @aguycalled |
I mean how would one pick which of the fiat currencies to use for the conversion from the main screen. |
User should be able to tap on the balance circle to switch between NAV or FIAT value.
Wallet can fetch the price from
https://api.coingecko.com/api/v3/simple/price?ids=nav-coin&vs_currencies=CURRENCY
, with currency being one of:User should be able to set the currency through the
Settings
screen.The text was updated successfully, but these errors were encountered: