iqlusioninc / liquidity-staking-module

Other
87 stars 29 forks source link

Fix bug in total stake calculation #108

Closed sampocs closed 1 year ago

sampocs commented 1 year ago

See #89 for more context and s/o @xlab for catching this!

Brief Changelog

sampocs commented 1 year ago

Do you guys think it makes more sense to have the bool as tokenizedShares and the check is

if !tokenizedShares {
  //  updatedTotal
}

Or as newDelegation and the check is

if newDelegation {
  // update total
}

Latter might be more clear

xlab commented 1 year ago

LGTM