-
Notifications
You must be signed in to change notification settings - Fork 44
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 stargate strategy #74
base: master
Are you sure you want to change the base?
Conversation
@@ -0,0 +1,10 @@ | |||
pragma solidity 0.5.16; | |||
|
|||
interface ICurvePoolV2 { |
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.
contracts/base/interface/curve/ICurvePoolV2.sol
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.
Just one comment on the getRewardPoolValues
function. The rest looks good good. Thanks!
return getUint256(_STARGATE_REWARD_PID_SLOT); | ||
} | ||
|
||
function getRewardPoolValues() public returns (uint256[] memory values) { |
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.
What is the use of this function?
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.
So this function is to get current pending STG amounts from the pool, it's mostly unused, just to see how much we can harvest, it can be view function I guess.
Do you wanna keep it or remove it?
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.
I think it can be removed. We won't use it. We estimate profit of doHardWork by simulating the doHardWork transaction.
Add stargate strategy