-
Notifications
You must be signed in to change notification settings - Fork 33
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
feature: Add Hashstack Base APY #156
Comments
Hi @akiraonstarknet can I work on this? ETA: 2 to 3 days. |
@Iwueseiter @ShantelPeters please share details of your implementation. (E.g. the files & functions you will modify) to achieve this. Also mention timeline. |
@ShantelPeters sorry to say this, but we cannot accept your request. Yesterday you had mentioned an incorrect approach and today's updated comment fully involves writing about StarkDefi instead of Hashstack. Third, you havent mentioned files or functions as well. Thank you, please try some other issue once they open. |
@Iwueseiter why to look at network tab, when I shared their APY calculation logic file? They use it in their defillama calcs. You could simply use that with some modifications relevant to us. |
Noted. I will look at the file. |
Is your feature request related to a problem? Please describe.
At present, hashstack pools listed do not have base APYs shown. We need to work on it to fix that.
Describe the solution you'd like
Use the logic used by hashstack to compute their apyBase here to embed logic into our platform. https://github.com/0xHashstack/yield-server/blob/8498cd35ba7e38a7d2333d186b5d3d94e5653e4f/src/adaptors/hashstack/index.js
The text was updated successfully, but these errors were encountered: