bpanel-org/bwallet

bpanel wallet plugin

bwallet's Issues

Proposal Approval Screen

If there are many inputs/outputs, the information rendered on the screen is incorrect.
Specifically the proposal amount is incorrect.

We need to dig into why this is

Add breadcrumbs in create UX

Right now the create UX isn't the best because there are no breadcrumbs and there are many options.

Add a way for users to know where they are going when they are creating new wallets

Input fields - which units

It isn't always clear which units are being used in the input fields.

• Be very explicit with which units are used
• Bonus: allow for dynamic switching of units

User experience notes

Suggestions for UI layout

• see #47

User Interface

• Welcome message: After seeing Buck's comment on #47 now I see that the message is intended to be more of a disclaimer, so maybe its best if it does just pop up once (modal?) on the first install

• Create a new wallet: where is the mnemonic seed?

• TX list: tx amount shows total bitcoins moved (input value) not the amount actually received by the user (i.e. spending from coinbase, tx amount is 50)

• wallet / account balance only displays confirmed transactions, would be helpful to see unconfirmed for instant gratification even if its 0-conf.

Banging on bwallet, I encountered a lot of errors that don't get shown to the user:

• Create new account in modal: account name can not have spaces, wallet password must be correct

• Multisig: n must be > 1 (see: bcoin-org/bmultisig#27)

Feature requests

• When using hardware, user should be redirected to https://localhost:5001 and check ssl

• Multisig wallet without hardware (arbitrary xpub, or locally hosted?)

nits

• not super stoked about lower case "btc" but that comes from actual bcoin pkg

• TX list: wallet and account aren't really needed in the table (because the tx list is already "inside" a selected wallet/account view) but that comes from bpanel-ui "tx list" component

errors I'm getting that my be my own fault (bad config?)...

• Your wallet undefined of undefined test2 will be initialized after all participants have joined ... undefined?

• new wallet not appearing in sidebar right away (must nav away and back to bwallet to see it)

• switching clients doesn't show correct wallets (either shows wallets for previous client or none at all). I'm testing bpanel/bwallet with simnet full, simnet SPV, and bcash regtest and switching clients.

Final Cosigner Join skips showing tokens

When the final cosigner joins, there is no redirect to the page for displaying the cosigner token.
The only way to get the token now is from looking at the requests in the chrome console, or using redux dev tools. This needs to be fixed

Address generation should support bech32 addresses

UI doesn't seem to have an option for choosing which address format you would like to generate. I think this could be a very useful feature to have, especially for clarity on distinguishing between BCH and BTC addresses.

Prevent calls to multisig server if it is not enabled

The front end will still make calls to the multisig server, even if it is not enabled.
There should be a check to make sure that this does not happen, since bwallet should work
fine with just the plain wallet server. It should also be smart if there is no wallet server
running at all

Balancing the needs of users

The learning curve in the blockchain space is very high for new users relative to the app space. New users may be scared off by in depth features, while power users may desire more control in their applications.

bPanel can fix this by having an advanced mode that toggles on/off in the settings, and plugin developers can look for advanced: true in the redux store to conditionally render certain ux

bmultisig dependency

Update bmultisig to latest version, or pull from github

This will fix the 2 of 3 bug we saw

Wallet view doesn't update when switching to diff wallet

This was introduced with the fix in e043a63 to stop the unmounting of the view. The view now does not get updated if you move to a different wallet (you're stuck on whatever the first one you pick is).

Steps to reproduce:

1. have more than one wallet
2. enter bwallet, go to one wallet.
3. Navigate to a different wallet.

Expected: UI should update for the new wallet
Actual: the url changes but the wallet is still the first one selected
Workaround- Navigate out of bwallet and back again to remount the view

Should be able to navigate directly to a subpage

Steps to reproduce:

1. navigate to /bwallet/overview
2. Refresh page
3. You'll be back on /bwallet

Expected behavior: should refresh back to /bwallet/overview

Gotchyas- I imagine this could cause problems for wallet pages since that information needs to be fetched first. It might be able to fetch based on the path, but if not, then can just reroute to overview if on a wallet page.