Luka Jeran
Luka Jeran
Instead of persisting and syncing everything, we should be more selective. Current state, by longterm-ness - theme, network, wallet, account, dapp (soon) - create tx, create wallet, import ledger, stake...
Updated  [nomnoml diagram source](https://nomnoml.com/#view/%5BCurrent%20state%2C%20by%20longterm-ness%20%7C%0Atheme%0Anetwork%0Awallet%0Aaccount%0Adapp%20(soon)%20%7C%0Acreate%20tx%0Acreate%20wallet%0Aimport%20ledger%0Astake%20%7C%0AfatalError%5D%0A%0A%5BEncrypted%20storage%20%7C%0Atheme%0Anetwork%20(partially)%0Awallet%0Aselected%20account%0Adapp%20(soon)%5D%0A%0A%5BExtension%20unlock%20background%20page%20%7C%0Apassword%20hash%0Aeverything%20in%20storage%0Acreate%20tx%20(conflict%20if%20multiple%20popups)%0AfatalError%0Await%20for%20tx%20status%20%26%20notify%20(soon)%0AURL%3F%5D%0A%0A%5BExtension%20popup%20%7C%0AURL%3F%5D%0A%0A%0A%5BEncrypted%20storage%5D%20-%3E%20%5BExtension%20unlock%20background%20page%5D%0A%5BExtension%20unlock%20background%20page%5D%20-%3E%20%5BExtension%20popup%5D%0A%0A%0A%5BWeb%20unlocked%20tab%20%7C%0Apassword%20hash%0Aeverything%20in%20storage%0Acreate%20tx%20(conflict%20if%20sync%20tabs)%0AfatalError%5D%0A%0A%5BSync%20multiple%20tabs%20%7C%0AInitial%20state%20sync%3A%20%7C%0Apassword%20hash%0Aeverything%20in%20storage%2C%20except%3A%0A-%20selected%20account%0A-%20(conflict%20-%20store%20selection%20in%20last%20tab)%0A%7C%0ASync%20every%20action%3A%20%7C%0Aunlock%2C%20lock%0Achange%20theme%0Achange%20network%0Aadd%20wallet%0Aupdate%20state.wallet%5C%5B*%5C%5D.balance%0A%5D%0A%0A%0A%5BEncrypted%20storage%5D%20-%3E%20%5BWeb%20unlocked%20tab%5D%0A%5BWeb%20unlocked%20tab%5D%20-%3E%20%5BSync%20multiple%20tabs%5D%0A%0A)
Deferred out of https://github.com/oasisprotocol/oasis-wallet-web/pull/975 - [x] test syncing tabs - [ ] auto lock timeout - [x] migrate domain: wallet.oasisprotocol.org -> wallet.oasis.io - [ ] password rules https://github.com/oasisprotocol/oasis-wallet-web/pull/975#issuecomment-1277373366 - [...
This will be more annoying after https://github.com/oasisprotocol/oasis-wallet-web/pull/975
Additionally, `qwertyuiopasdfghjkl` test is odd. Use e.g. oasis1qq2vzcvxn0js5unsch5me2xz4kr43vcasv0d5eq4 (used in PrettyAddress test)
- imo we don't have to duplicate amount in title. So it can look like this: "Type(title) addr, date, amount" and not "Type+amount, addr, date, amount". It's easier to "scan"...
- Removing amounts out of title: I agree for transfers, especially of less pretty amounts. But I like the amount in additional context when setting allowance - maybe the amount...
Discussed with Nikola that we could also show amounts with +/- prefix (except for transactions like set allowance)
About roothash.ExecutorCommit: ExecutorCommit contains events relating to your account. Some transactions (e.g. withdrawing from paratime) SOMETIMES only emit an event, without a standalone transaction. Oasisscan also only indexes Emerald paratime...