Skip to content
This repository has been archived by the owner on Jun 16, 2022. It is now read-only.

[Android] - Main Balance and Ethereum Tokens Balance incorrect in AUD #1697

Open
pwrfix opened this issue May 6, 2021 · 2 comments
Open
Assignees

Comments

@pwrfix
Copy link

pwrfix commented May 6, 2021

Ledger Live Version and Operating System
Tested on Ledger Live 2.26.0 (arm64-v8a 156)
Phone version: Samsung S20, latest android

Expected behavior

  • TEL Coin balance correctly displayed in AUD
  • Account total balance (main screen) correctly matches individual account balances sum

Actual behavior

  • TEL Coin balance moves in wild swings not correlating to current price. Seems to sometimes display total asset balance in USD, even though it's AUD
  • Account balance on main screen does not sum correctly to individual accounts in portfolio

Steps to reproduce the behavior
Sporadically (more often than not), balances in AUD for TEL Coin (Ethereum) do not calculate correctly or update at all, even when price is moving.

Clearing cache doesn't seem to correct - it's as if the underlying USD->AUD conversion is failing and/or not returning the correct currency exchange * number of assets * USD price.

Account balance on main screen (after refresh) provides a sum not the same as the individual accounts in the portfolio screen (after refresh). Seems to completely ignore ETH tokens in account balance on main screen.

@gre
Copy link
Contributor

gre commented May 6, 2021

Hi @pwrfix and thanks for creating the issue.

It sounds like something we have recently fixed (via #1681) and that will be released in our next 2.27 release.

The bug was due to a cache on client side that was retaining the price between two different ranges. at the same time there are work to make our API better when there are volatility.

Let's keep this issue open and i can ping you when 2.27 is out. (right now it's actually a Beta on Android)

@gre gre self-assigned this May 6, 2021
@pwrfix
Copy link
Author

pwrfix commented May 7, 2021

Thanks @gre. I'll await the new version.

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

No branches or pull requests

2 participants