-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[dapps] Show CurrentBalance in dApps #6
Comments
Is there any work to be done in the |
@snario that is correct. We just need to show it :) |
Just in terms of communicating consistently to the user, I think we want to move towards a picture where the user's channel balance is just thought of as part of their wallet balance. The UI for a specific app needs to show the money committed to that app, and they do need to know their channel "limit" when opening an app. But we should figure out a clear way of communicating all this consistently so the user doesn't get confused with all their different balances. I'll add it to the "vocabulary systematization" project. |
@emansipater I agree. Since MM UI is usually hidden and transactions don't need to be signed. Users of a dApp can quickly lose track of their balance. I think we should teach that best practice for State Channel dApps is to include the balance in the UI. |
This is partially being addressed by @ebryn in his Wallet UI spike and @cf19drofxots with the work being done with Connext. |
@snario Can you elaborate on how this is being addressed? |
Makes sense. Can we subdivide this into two issues then, one for TTT and one for High Roller? They need to call the |
The current architecture was built with the fact that the PG UI shows the
CurrentBalance
of theUser
at all times while the dApps live inside an iFrame.With the new architecture the PG UI lives inside of MM, which is usually closed or in a different tab, and the dApp lives standalone.
This creates a UX issue where a person playing/using a dApp doesn't know their
CurrentBalance
of their funds in themultisig
This issue suggests adding UX for
CurrentBalance
in the dApps: High Roller and Tic Tac ToeThe text was updated successfully, but these errors were encountered: