Skip to content

Conversation

@tannishmango
Copy link

@tannishmango tannishmango commented Mar 24, 2022

Fixes #OE-10

PR Checklist

  • Tests
  • Documentation
  • Changelog entry

@tannishmango tannishmango changed the title initial commit alpha homora v2 on avax initial commit Mar 24, 2022
@uwe
Copy link
Contributor

uwe commented Mar 24, 2022

Module 'src/utils/constants' has no exported member 'homoraBankAddress'.

@uwe
Copy link
Contributor

uwe commented Mar 24, 2022

And constants please in all upper case (with underscores).

@uwe
Copy link
Contributor

uwe commented Mar 27, 2022

config/avalanche-dev.json should not be committed! It is in .gitignore.

Copy link
Contributor

@uwe uwe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On the Lend pools it is unclear how we could calculate the APY. For example "AVAX" shows 3.81% Earn APY, 1.60% Avax APR and 1.83 ALPHA.e APR.
Can you please explain that for one pool and current data from the web site?

]
*/

export const poolRegistry: Array<string[]> = [
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where is this coming from? Can't we get this via web3 calls (or by listening to some events)?
The web site lists 18 pools, but here are only 16.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will commit change to this static list soon, going to make it dynamic

chef: String!
}

type AlphaHomoraV2FarmData @entity {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How can we get the TVL for a farm?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TVL for farm data be obtained by multiplying the LP tokens locked value by the usd value of the lp token

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@d-fda is this a problem? I think we can get prices for "real" tokens (like "AVAX"), but not for LP tokens.

@uwe uwe changed the title alpha homora v2 on avax initial commit Alpha Homora v2 on Avalanche #OE-10 Apr 7, 2022
chef: String!
}

type AlphaHomoraV2FarmData @entity {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@d-fda is this a problem? I think we can get prices for "real" tokens (like "AVAX"), but not for LP tokens.

} else {
decimals = decimalsResult.value;
}
decimals = BigInt.fromI32(decimals).toI32();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this needed? What does it do?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

accidentally left line 45 in, is not needed.

make sure multiplication is done before subtracting reward debt
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants