-
Notifications
You must be signed in to change notification settings - Fork 18
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
Invoke contracts #151
Invoke contracts #151
Conversation
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.
Left a few comments!
4e9ae2a
to
3b5aaa2
Compare
48e7884
to
1e9a6b5
Compare
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.
Just a few minor comments. I have to be honest. It is not easy to review hahah. Something I noticed from zemu images, and I am not sure is are we saying "Transfer" when we are clear signing a well-known contract? I didn't see it on the images.
1e2eda0
to
fa8268e
Compare
fa8268e
to
6f03b2b
Compare
4ac8a96
to
7dd1102
Compare
Improve display of params when EVM Invoke call is ERC20 transfer
Explicit check that users cannot make blindsign with normal mode
Bugfix ERC20 transfer
Improve format from amounts (token + decimals for several fields)
Update dependencies
🔗 zboto Link