-
Notifications
You must be signed in to change notification settings - Fork 982
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
Update status js api according to latest changes in web3 opt-in EIP #5648
Conversation
Desktop Jenkins build job started at https://ci.status.im/job/status-react/job/desktop/job/pull-requests/job/PR-5648/1/ |
Branch: PR-5648, Android build uploaded to diawi: https://i.diawi.com/pkUxtj |
Hey @flexsurfer! What tests it makes sense to do here? Anything else besides transactions in dapps (including Status Test DApp) and basic smoke test? |
@lukaszfryc just check if you can access contact code in Test Dapp, that should be enough |
Desktop Jenkins build job started at https://ci.status.im/job/status-react/job/desktop/job/pull-requests/job/PR-5648/2/ |
Desktop Jenkins build was failed or cancelled. More details at https://ci.status.im/job/status-react/job/desktop/job/pull-requests/job/PR-5648/2/ |
81% of end-end tests have passed
Failed tests (11)Click to expand
Passed tests (47)Click to expand
|
LGTM. Autotests reviewed and confirmed not this PR issue and will be addressed separately. |
Signed-off-by: Andrey Shovkoplyas <motor4ik@gmail.com>
813f15f
to
b754092
Compare
Docs: https://github.com/status-im/docs.status.im/pull/44