Fiat value not properly fetched (possibly 3rd party API fault). Suggestion for workaround.

[onvisions]

The Fiat value of Coins/Tokens is not properly fetched by third party API (CoinGecko) which leads to following bugs observed in the wallet extension:

1. The Total portfolio value of all accounts is displayed as "0" although the user has tokens and coins in various accounts.
2. Fiat values are displayed as "0" in the transactions list.
3. Fiat values of token amounts in the token selector (Sending or Receiving flow) are not displayed.

Suggestion: if the problem is with CoinGecko API we may implement logic to display the latest successfully fetched data about the asset price if it's no older than 10 mins (the exact period to be determined). In case data is older we should hide the fiat value until we can fetch properly. IMO It's more frustrating and confusing to see balance $0.00 instead of not seeing any value at all.

[peronczyk]

As we don't have any control if CoinGecko API call fails I agree that the only solution is to cache the previous values. I have implemented the local storage caching. The task is ready to review and test @Liubov-crypto .

In order to test it you can try to block the CoinGecko url in your browser after opening the app for the 1st time and then try to opn/close it again.

[Liubov-crypto]

After I blocked the CoinGecko URL and reloaded the app, it will show the total amount, but will not show the amount for each transaction, such as ETH. I don't know if this is correct, but the same is on product.

the rest is LGTM

I've registered mentioned issue separately.

[Liubov-crypto]

LGTM