search

trustlines-protocol / relay

MIT License
8 stars 7 forks source link

No longer push event information for balance updates from filter #577

Closed cducrest closed 3 years ago

cducrest commented 3 years ago

Use ethindex updates instead. The updates were incorrect because they were using information taken from the graph before it was updated (the filter being triggered faster). Now react on graph feed to publish events about trustline updates and balance updates

codecov[bot] commented 3 years ago

Codecov Report

Merging #577 (c06efc1) into master (ec65254) will increase coverage by 0.02%. The diff coverage is 94.44%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #577      +/-   ##
==========================================
+ Coverage   85.84%   85.87%   +0.02%     
==========================================
  Files          56       56              
  Lines        4791     4799       +8     
==========================================
+ Hits         4113     4121       +8     
  Misses        678      678
Impacted Files Coverage Δ
src/relay/blockchain/currency_network_proxy.py 98.38% <ø> (-0.10%) :arrow_down:
src/relay/network_graph/graph.py 90.32% <ø> (ø)
src/relay/relay.py 75.60% <94.44%> (-1.05%) :arrow_down:
src/relay/blockchain/proxy.py 87.71% <0.00%> (ø)
src/relay/ethindex_db/ethindex_db.py 85.65% <0.00%> (+0.44%) :arrow_up:
src/relay/blockchain/delegate.py 90.74% <0.00%> (+0.92%) :arrow_up:
src/relay/api/resources.py 88.22% <0.00%> (+1.33%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update ec65254...c06efc1. Read the comment docs.