Fix: stake limit resume handling

lidofinance / core

Lido DAO smart contracts

https://lido.fi

GNU General Public License v3.0

382 stars 194 forks source link

Fix: stake limit resume handling #721

Closed TheDZhon closed 1 year ago

TheDZhon commented 1 year ago

Test cases are in test/0.4.24/staking-limit.test.js

TheDZhon commented 1 year ago

Fine, but can optionally use some work on naming and comments. As for me, prevStakeLimit is really misleading name here and can use some rebranding.

Agree, added a few comments, but decided to not proceed with a large-scale refactoring