saddle-finance / saddle-frontend

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

Bug fix on network change #1156

Closed arthur-zhuk closed 2 years ago

arthur-zhuk commented 2 years ago

There is a scenario when changing wallets from Opt -> Arb causes this type error to occur.

hard to reproduce, bit of a race condtition

codecov[bot] commented 2 years ago

Codecov Report

Merging #1156 (deb676e) into master (29c5125) will not change coverage. The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master    #1156   +/-   ##
=======================================
  Coverage   10.86%   10.86%           
=======================================
  Files         154      154           
  Lines        5642     5642           
  Branches     1788     1788           
=======================================
  Hits          613      613           
  Misses       5027     5027           
  Partials        2        2           
Impacted Files Coverage Δ
src/providers/RewardsBalancesProvider.tsx 0.00% <0.00%> (ø)

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 29c5125...deb676e. Read the comment docs.