-
Notifications
You must be signed in to change notification settings - Fork 43
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
gas limit fix for RSK chain #290
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.
@vbaranov any idea why ci/circleci: test-e2e-chrome
is failing?
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.
Changes looks good to me. Once errors from ci/circleci: test-e2e-chrome
are fixed, we can merge this
I removed blinking tests. |
Build on top of #289
For simple coin transfer gas limit was calculated incorrectly.
Missing
code === '0x0
added to check if this simple coin transfer or not.