search

trezor / trezor-suite

Trezor Suite Monorepo
https://trezor.io/trezor-suite
Other
704 stars 248 forks source link

Portfolio graph does not work with imported testnet asset #8481

Closed PeKne closed 1 year ago

PeKne commented 1 year ago

Current behaviour: If there is any testnet account imported, then the portfolio graph shows a zero balance and error message.

Expected behaviour: The testnet does not affects the portfolio graph at all and it shows balance of other imported assets.

Screenshot of the bug

Image

bosomt commented 1 year ago

Account detail looks as expected. Overall Home graph is not rendered properly / not disabled when only testnet coins enabled/

Tested in 23.5.1 (425) on Android os

Screenshot_20230524_112040_Dev-TrezorSuite

Screenshot_20230524_112023_Dev-TrezorSuite

PeKne commented 1 year ago

@mnuky How should the graph look if there is no other asset imported than a testnet?

PeKne commented 1 year ago

@bosomt The issue was fixed with https://github.com/trezor/trezor-suite/pull/8587. If only testnet accounts are imported, the graph shows an error message now.

Please test again.

bosomt commented 1 year ago

QA NOK

no idea what happened buy latest dev version 430 is now showing graphl loading and fiat value of test account ;)

image

bosomt commented 1 year ago

QA OK

tested on Android dev version 23.5.1 (433)