-
Notifications
You must be signed in to change notification settings - Fork 5
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
Feat/stos #119
Feat/stos #119
Conversation
This is in line with changes to the smart contracts which will allow Tiered STOs to raise funds in any denomination BREAKING CHANGE: entity `UsdTieredSto` rernamed to `TieredSto`, enum `StoType.UsdTiered` changed to `StoType.Tiered`
Also add internal support for future methods and resolver arrays
Launch functions and procedure params were named differently from the entity properties BREAKING CHANGE: rename STO entity properties for consistency
Also fixed some wrong procedure types
message: `Cannot invest on behalf of ${beneficiary} because this STO doesn't allow beneficial investments`, | ||
}); | ||
} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could add whitelist validation here in the future.
src/entities/Sto.ts
Outdated
this.startDate = startDate; | ||
this.endDate = endDate; | ||
this.raisedFundsWallet = raisedFundsWallet; | ||
this.unsoldTokensWallet = unsoldTokensWallet; | ||
this.raisedAmount = raisedAmount; | ||
this.soldTokensAmount = soldTokensAmount; | ||
this.investorAmount = investorAmount; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we change this to investorCount
everywhere?
Removed `investments` in favor of a `getInvestments` function. Changed `investorAmount` to `investorCount` BREAKING CHANGE: Removed `investments` in favor of a `getInvestments` function. Changed `investorAmount` to `investorCount`
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great. Good job. The only pending task is to change CappedSTO
to SimpleSTO
everywhere it is public.
All relevant types and enums were changed as well BREAKING CHANGE: rename CappedSto to SimpleSto
🎉 This PR is included in version 2.0.1-beta.66 🎉 The release is available on: Your semantic-release bot 📦🚀 |
No description provided.