semuxproject / semux-keeper-extension Goto Github PK

Show delegate names. Add search by delegates (ie: select2 or similar)

When user tries to import already existing account via privkey or json file, we should notify him that account already exists and don't create duplicates in the wallet.

Show "welcome" page. Require user password to unlock app.

Can someone provide an instruction?

Right now user can create new accounts with a different passwords. This may be very confusing UX because users will have to backup both passwords and private keys. Maybe we should force user to use same password as he used for created/imported account 0.

Since we don't store passwords, the flow can be as follows:

1. Ask for a password to create new account
2. Verify if this password can successfully decrypt address 0 from storage.
3. Throw error "Invalid Password" if user tried to use a new password.

Thoughts?

Show pending outgoing transactions in the list because right now it's not clear if the TX was successfully broadcasted to the network.
Order transactions by date: most recent ones on the top.

Error messages, notifications, inputs, buttons

There are some duplicated pieces of code which should be moved to helpers.
All promises must reject in case of errors.
try catch is required for all requests to remote servers (semux node, exchanges, other)
Move all hardcoded constants (URLs, min fee, ..) to a config file.
Code review.