Closed 3esmit closed 6 months ago
This looks very nice to me!
Can we create issues for the items you've added in the "todo" commit?
Yes, I didnt fixed them here because the fix is not that simple. I mean, I could just multiply by 100 and divide it back, however there are other ways of fixing them, and I want to have a deeper insight on that solutions,
Description
Replaces #52
Renames fields to more descritive variable names:
New field:
processEpoch is now called at first of every external function that would need it, as a modifier.
Fix in migration of processAccount, send only the difference of generated MP to new contract.
Now with code reuse for math formulas
Mint Intial (Bonus) MPs now use the same formula as the regular multiplier points, but deltaTime is lockedTime.
Fixed the Lock and Mint function about locked funds. Now it mints correctly for increased time locked, and increased balance in remaining time locked. Fixes #13
Changed how lock function works: Now lock time can only increase (previously it reverted), now the parameter value represents time increased, and therefore this case does not exist anymore.
Accounts are properly initialized at first stake (Fixes #49)
Checklist
Ensure you completed all of the steps below before submitting your pull request:
forge snapshot
?pnpm lint
?forge test
?pnpm verify
?