-
Notifications
You must be signed in to change notification settings - Fork 840
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: add hydt yield adapter #1071
Conversation
The hydt-protocol adapter exports pools: Test Suites: 1 passed, 1 total
|
hi, ty for adding. it seems like you are adding up all components under apyBase? |
Hi @slasher125, thank you for the response. If I understand correctly, any tokens being cut as fees and then being redistributed to the users as rewards would go to apyBase and if not then they go to apyReward. If that is the case then there is a slight error on my part. All of our rewards are minted directly to the user as incentives and hence should go to apyRewards. Should I submit a new PR or can you make the changes by swapping the values for apyBase and apyReward at Line#275 and Line#356 (kindly change the variable name as well from "baseAPY" to "rewardAPY" declared at Line#344)? The front end is having issues right now and is not showing any APY. I'm currently looking into that as well. |
hey, yes exactly. yes, just commit the changes to this pr |
@slasher125 Changes committed. Kindly have a look if the commit is done correctly or not. |
The hydt-protocol adapter exports pools: Test Suites: 1 passed, 1 total
|
No description provided.