-
Notifications
You must be signed in to change notification settings - Fork 17
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
Add transaction confirmation screen #22
Comments
@BrandtH22 is this still an issue, now that the coins will be displayed as pending a transaction? In other words, was the problem lack of UI feedback or lack of confirmation? |
I think the transaction confirmation that does appear would be nice to have the coinIDs that will be used in the transaction but that could be me just liking to have more data than needed |
I think maybe having a generic transaction confirmation screen that shows all inputs and outputs would make sense. So it builds the full spend bundle, displays it, then adds it to the transactions table if confirmed. This could be reused for WalletConnect and offers |
Resolved in #61 :) |
Add transaction confirmation screen (currently reverts back to token list but hangs on the transaction screen while submitting)
The text was updated successfully, but these errors were encountered: