I just did some more testing with the wallet, and here are a couple of issues I found:
Thank you for doing so, first of all.
- When you refresh the site while you're on the Send/receive or LN receive pages, you'll get an error saying that the file does not exist.
- The site asks you to re-enter your password when you simply refresh the transactions page.
Both of these issues are actually intended behaviour, though it can probably be repackaged to look a bit more professional.
- When you refresh the site while you're on the Send/receive or LN receive pages, you'll get an error saying that the file does not exist.
I see. This is because you're asking the host (github) for a certain URL that only existed client-side (SPA). We should just automatically redirect you back to the "login page" to avoid any confusion.
- The site asks you to re-enter your password when you simply refresh the transactions page.
This is also intended behaviour; the wallet is not really supposed to be "refreshed". If you have just made a transaction it should automatically redirect you to the transaction status. Refreshing the transaction page will not actually fetch any new data. Perhaps it does display previously unseen states that were already stored locally but not visible yet, however switching between the send/receive/transaction tab should make these new entries visible -- there would be no need to refresh. I guess this should perhaps be made more clear?
I'm also not sure what the "Check" and "sync" buttons are supposed to do since nothing is happening when I click them.
The "check" button sends a request to a blockchain explorer to discover if there have been any new transactions made to said address - if so - the wallet will ask the custodian to accredit the deposit.
I agree with you that there are two potential issues / improvements. We could remove the button on the /receive/ tab and automatically check the latest address to see if it received any new deposits, (but keep it on /addresses/ as that would become resource-intensive)
And secondly we should probably redirect users to the deposit once it is found. Currently after clicking "Check" you should switch back to the transactions tab to see the new transaction item and the corresponding information, which is admittedly a bit confusing. (No page refresh is needed though)
The sync button checks for any missing deposits, withdrawals, or unfinished states (unclaimed) by querying the custodian with your deposit addresses + unspent coins. You can try and see for yourself by taking your current seed, restore, and click sync. The transactions tab should automatically be populated with all of your previous transactions, and attempt to finalize them.