-
Notifications
You must be signed in to change notification settings - Fork 19
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
Add flow for depositing into the pool #29
Conversation
contracts/Pool.sol
Outdated
this.previewDeposit(assets), | ||
this.maxDeposit(receiver) | ||
); | ||
_mint(receiver, shares); |
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.
Since _mint is internal, the library can't call it. Not sure a good pattern for this; ideally we could move everything to the library for testability; not crazy about having the boundary here.
atState(PoolLifeCycleState.Active) | ||
returns (uint256 shares) | ||
{ | ||
// TODO: check lender ACLs for both msg.sender and receiver |
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.
This TODO can be part of the PermissionedPool logic (once it lands)
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.
Yep, will rebase mine + update it once that lands!
Left one question below in-line