Skip to content
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

Refactor/call params #8233

Closed
wants to merge 1 commit into from
Closed

Conversation

yenda
Copy link
Contributor

@yenda yenda commented May 20, 2019

  • use json-rpc/eth-call and wallet/eth-transaction-call everywhere
  • move all conversions to abi-spec

The goal of this PR is to unify the way we interact with contracts, so that future improvements can be made and impact all the callees. json-rpc/eth-call is for read only calls and wallet/eth-transaction-call is for transnational calls.

status: ready

@yenda yenda added the wallet label May 20, 2019
@yenda yenda requested review from antdanchenko, churik, jakubgs and a team as code owners May 20, 2019 03:01
@yenda yenda self-assigned this May 20, 2019
@status-github-bot
Copy link

status-github-bot bot commented May 20, 2019

Pull Request Checklist

  • Have you updated the documentation, if impacted (e.g. docs.status.im)?

@yenda yenda changed the base branch from develop to wallet/remove-web3 May 20, 2019 03:02
@status-im-auto
Copy link
Member

status-im-auto commented May 20, 2019

Jenkins Builds

Click to see older builds (17)
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ eb7dbee #2 2019-05-20 03:20:14 ~17 min macos 📦 dmg
✔️ eb7dbee #2 2019-05-20 03:20:33 ~18 min linux 📦 App
✔️ eb7dbee #2 2019-05-20 03:21:29 ~19 min windows 📦 exe
✔️ d524f26 #4 2019-05-20 03:37:55 ~27 min linux 📦 App
✔️ d524f26 #4 2019-05-20 03:39:01 ~28 min android-e2e 📦 apk
✔️ d524f26 #4 2019-05-20 03:39:02 ~28 min macos 📦 dmg
✔️ d524f26 #4 2019-05-20 03:43:24 ~32 min android 📦 apk
✔️ d524f26 #4 2019-05-20 03:44:09 ~33 min ios 📦 ipa
✔️ d524f26 #4 2019-05-20 03:48:26 ~37 min windows 📦 exe
✔️ 658ef4d #5 2019-05-20 12:40:30 ~24 min macos 📦 dmg
✔️ 658ef4d #5 2019-05-20 12:41:31 ~25 min android-e2e 📦 apk
✔️ 658ef4d #5 2019-05-20 12:42:59 ~26 min linux 📦 App
✔️ 658ef4d #5 2019-05-20 12:44:15 ~27 min ios 📦 ipa
✔️ 658ef4d #5 2019-05-20 12:44:27 ~28 min windows 📦 exe
✔️ 658ef4d #5 2019-05-20 12:50:53 ~34 min android 📦 apk
d8ce0a4 #6 2019-05-20 13:12:26 ~3 min ios 📄 log
d8ce0a4 #6 2019-05-20 13:13:11 ~3 min macos 📄 log
Commit #️⃣ Finished (UTC) Duration Platform Result
7f09650 #7 2019-05-20 13:20:38 ~3 min ios 📄 log
7f09650 #7 2019-05-20 13:21:13 ~4 min macos 📄 log
7f09650 #7 2019-05-20 13:32:49 ~15 min linux 📄 log
7f09650 #7 2019-05-20 13:34:09 ~17 min android 📄 log
7f09650 #7 2019-05-20 13:36:34 ~19 min android-e2e 📄 log
7f09650 #7 2019-05-20 13:37:15 ~20 min windows 📄 log
2482706 #10 2019-05-20 13:57:40 ~16 min ios 📄 log
✔️ 2482706 #10 2019-05-20 14:17:07 ~35 min macos 📦 dmg
✔️ 2482706 #10 2019-05-20 14:31:26 ~49 min android 📦 apk
✔️ 2482706 #10 2019-05-20 14:31:32 ~50 min android-e2e 📦 apk
✔️ 2482706 #11 2019-05-20 14:35:19 ~23 min ios 📦 ipa
✔️ 2482706 #10 2019-05-20 14:36:27 ~54 min linux 📦 App
✔️ 2482706 #9 2019-05-20 14:37:28 ~55 min windows 📦 exe

@yenda yenda force-pushed the refactor/call-params branch 2 times, most recently from d8ce0a4 to 7f09650 Compare May 20, 2019 13:16
@yenda yenda force-pushed the refactor/call-params branch 3 times, most recently from b8869a3 to 2482706 Compare May 20, 2019 13:41
@asemiankevich
Copy link
Contributor

@yenda all builds are failed

- use `json-rpc/eth-call` and `json-rpc/eth-transaction-call`
everywhere
- move all conversions to abi-spec
@yenda yenda closed this May 20, 2019
@jakubgs jakubgs deleted the refactor/call-params branch May 12, 2020 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

5 participants