search

trezor / trezor-suite

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

Performance issues of Graph on mobile #13821

Closed Nodonisko closed 1 month ago

Nodonisko commented 2 months ago

First loading could take a long time (depends on how many transactions user has) - we should consider some kind of progress bar

App become unresponsive during graph loading (Sometime it's impossible to navigate to Settings while graph is still loading)

matejkriz commented 2 months ago
  1. ETC doesn't work correctly 0x500910C4c42F1F2eeAF7E36d0c6f41e0F8858d5f not locally nor from blockbook. It's showing negative values - very low prio right now

moved to https://github.com/trezor/trezor-suite/issues/13987

matejkriz commented 2 months ago
  1. It can go to negative balance (could be reproduced using all seed) - discovered by @vytick

moved to https://github.com/trezor/trezor-suite/issues/13988

matejkriz commented 2 months ago

related https://github.com/trezor/trezor-suite/issues/13019

matejkriz commented 2 months ago
  1. Pending transactions are added to graph balance incorrectly, because they will be added for whole graph line not for just last point or not at all.

moved to https://github.com/trezor/trezor-suite/issues/14003

matejkriz commented 2 months ago
  1. Staking is not reflected in graph

Reflect staking in mobile app has it's own issue https://github.com/trezor/trezor-suite/issues/12187

matejkriz commented 1 month ago

Closing this now as it seems to be mostly resolved. Moving to Approved directly, doesn't make sense to test separately.