-
Notifications
You must be signed in to change notification settings - Fork 985
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
Feature/token transaction signals #8184
Conversation
Pull Request Checklist
|
Jenkins BuildsClick to see older builds (27)
|
f87783c
to
344b027
Compare
b2431aa
to
f63bf24
Compare
75eb403
to
72a3c1c
Compare
72a3c1c
to
111bc0e
Compare
@jeluard @flexsurfer @siphiuel please review this PR so it can go to testing |
(fn [response] | ||
(if (string/blank? response) | ||
(log/warn :web3-response-error) | ||
(callback (-> (.parse js/JSON response) |
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.
json->clj
100% of end-end tests have passed
Passed tests (49)Click to expand |
111bc0e
to
f9a126e
Compare
@flexsurfer @churik I propose to merge these wallet related PR into a meta wallet PR so that they are code reviewed in small iterations and can be QA tested all at once since they are all related to the transaction history |
- removes fetching of last 100000 blocks of token transfers from the wallet pull loop - fetches the last 100000 blocks of token transfers at startup - replaces pulling by subscriptions to ethlogs for token transfers
f9a126e
to
e10d59e
Compare
@yenda so it is not meta PR yet, yes? |
I just picked it up from |
part of #8151
the wallet pull loop
Testing
Impacts: ERC20 token transfers list (not token transfers themselves, only the refresh rate of their history)
Tested on Android:
satoshi document engage inflict goddess auction rule unfair bid next buddy shy
Expected behavior:
status: ready