Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

change min bond to be relative to IRO funds raised in case such exists #1577

Closed
omritoptix opened this issue Nov 29, 2024 · 1 comment · Fixed by #1579
Closed

change min bond to be relative to IRO funds raised in case such exists #1577

omritoptix opened this issue Nov 29, 2024 · 1 comment · Fixed by #1579
Assignees

Comments

@omritoptix
Copy link
Contributor

currently we enfoce 100DYM min bond.
this makes sense for rollapps without IRO or rollapps who raised small amount.
however in case of IRO raising significant funds, this is not enough to prevent attempt of malicious behavior.
We want to change it to be min(IRO_DYM_RAISED/bond_factor, 100DYM) where by default bond_factor = 100.

for example, of IRO_DYM_RAISED = 100kDYM, the min bond will be 1k DYM.
similariliy if IRO_DYM_RAISED = 9KDYM, min bond will be 100DYM.

This bond factor should be a param.

@omritoptix
Copy link
Contributor Author

updated impl to make the bond per rollapp with enforced min but it can be chosen by rollapp creator

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 a pull request may close this issue.

2 participants