Skip to content

Conversation

@cygnusv
Copy link
Member

@cygnusv cygnusv commented Aug 18, 2022

Change the staking pre-requisite from amount > minStake to amount >= minStake (Fixes #90)

vzotova
vzotova previously approved these changes Aug 18, 2022
@derekpierre
Copy link
Member

Does the Council still own TokenStaking? If so, when this update gets deployed, presumably to be complete the council needs to reset the minimum amount to be 40,000 T instead of the 39,999.9999 T we did previously via https://etherscan.io/tx/0xd87665e395a2f934abc4bb7d29c5c1da86429c6cf7856e59dfb090f05c203f54#eventlog?

@cygnusv
Copy link
Member Author

cygnusv commented Aug 18, 2022

Does the Council still own TokenStaking? If so, when this update gets deployed, presumably to be complete the council needs to reset the minimum amount to be 40,000 T instead of the 39,999.9999 T we did previously via https://etherscan.io/tx/0xd87665e395a2f934abc4bb7d29c5c1da86429c6cf7856e59dfb090f05c203f54#eventlog?

Yes to both things!

theref
theref previously approved these changes Aug 18, 2022
manumonti
manumonti previously approved these changes Aug 19, 2022
Copy link
Member

@manumonti manumonti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! ✌️

pdyraga
pdyraga previously approved these changes Aug 24, 2022
@pdyraga pdyraga enabled auto-merge August 24, 2022 08:46
@pdyraga pdyraga merged commit 0de3da5 into main Aug 24, 2022
@pdyraga pdyraga deleted the minstake branch August 24, 2022 11:37
@pdyraga pdyraga added this to the v1.2.0 milestone Sep 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow stake amount equal to min stake

6 participants