saddle-finance / saddle-frontend

An open source UI for saddle.finance 🤠
https://saddle.exchange
54 stars 58 forks source link

Remove gaugeToPoolAddress call in Gauges state #1260

Closed nguyenaiden closed 2 years ago

nguyenaiden commented 2 years ago

Description: In preparation for retrieving gauges while on side chain, we're aiming to reduce the reliance on gaugeHelperContract as it doesn't exist on side chain. We no longer need gaugeHelperContract.gaugeToPoolAddress() to match a gauge to a pool. We now use the BasicPools context and retrieve the information from there. With the current state on mainnet, this will reduce 1 large multicall step.

codecov[bot] commented 2 years ago

Codecov Report

Merging #1260 (97e522a) into master (5a6eb40) will increase coverage by 0.00%. The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master    #1260   +/-   ##
=======================================
  Coverage   10.24%   10.24%           
=======================================
  Files         162      162           
  Lines        6122     6119    -3     
  Branches     2020     2019    -1     
=======================================
  Hits          627      627           
+ Misses       5493     5490    -3     
  Partials        2        2           
Impacted Files Coverage Δ
src/utils/gauges.ts 0.88% <0.00%> (+0.02%) :arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more