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
signer v2 #10
signer v2 #10
Conversation
import "unsafe" | ||
|
||
|
||
type TransactioBatchSummary struct { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add comments here? For example, what are the valid transactionType? what is a denomination.
Does these types have to be string because of FFI?
@Kevingislason feel free to merge it if you think it is ready to go! |
Rust code is not building anymore due to changes to Edit: just kidding, needed to cargo update |
@Kevingislason There were some feature changes that I also pinged you on in another PR, related to |
Some important supplemental updates to the MVP version of signer:
-Display transaction info when confirming transactions
-Allow for mnemonic recovery
-Improve UI feedback
-Simplified and modestly improved UI styling
-Improve error handling in Rust code
todo: change manta-api branch to
manta
after mergingsigner_batching_auxiliary