saddle-finance / saddle-frontend

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

Add any missing PIDs #1253

Closed penandlim closed 2 years ago

penandlim commented 2 years ago

Went through the index file and noticed some of the PIDs dont follow the existing formats.

codecov[bot] commented 2 years ago

Codecov Report

Merging #1253 (02a96fa) into master (be0ad26) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1253   +/-   ##
=======================================
  Coverage   10.26%   10.26%           
=======================================
  Files         162      162           
  Lines        6091     6091           
  Branches     2008     2008           
=======================================
  Hits          625      625           
  Misses       5464     5464           
  Partials        2        2           
Impacted Files Coverage Δ
src/constants/index.ts 96.64% <ø> (ø)

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

hammeiam commented 2 years ago

Does arbusdv2 PID 7 also need to be added to the second set of PIDs?

penandlim commented 2 years ago

@hammeiam already added as FRAX_ARB_USD_SWAP_V2_ADDRESSES i believe.

arthur-zhuk commented 2 years ago
image

the only key interacting with minichefPids is poolAddress.