Skip to content
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 GyroEV2 pool #71

Merged
merged 18 commits into from
Jun 27, 2023
Merged

Add GyroEV2 pool #71

merged 18 commits into from
Jun 27, 2023

Conversation

brunoguerios
Copy link
Member

@brunoguerios brunoguerios commented Jun 19, 2023

  • Add GyroEV2 pool by extending GyroE pools to handle token rates

Note:

  • this implementation defaults GyroE v1 pools to have token rates of 1 and reuses same pool logic
  • this PR adds integration tests that run on Polygon, so it needs POLYGON_RPC_URL as env var (and it should also be added to github secrets in order to have PR checks working as expected)

@brunoguerios brunoguerios self-assigned this Jun 19, 2023
@brunoguerios brunoguerios changed the title Add GyroEV2 pool Add GyroEV2 pool (WIP) Jun 19, 2023
@brunoguerios brunoguerios marked this pull request as draft June 19, 2023 20:10
@brunoguerios brunoguerios changed the title Add GyroEV2 pool (WIP) Add GyroEV2 pool Jun 20, 2023
@brunoguerios brunoguerios marked this pull request as ready for review June 21, 2023 15:13
Copy link
Member

@johngrantuk johngrantuk left a comment

Choose a reason for hiding this comment

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

I think the onChainPoolDataEnricher needs to be updated to get the GyroE V2 tokenRates?

The current poolDataQuery helper doesn't support that. Could be done using a separate mulitcall like this. It would be more convenient if we updated the poolDataQuery helper but not sure if that's a reasonable approach?

@brunoguerios brunoguerios merged commit 75774e4 into feat/gyroE-pool Jun 27, 2023
@brunoguerios brunoguerios deleted the feat/gyroEv2 branch June 27, 2023 17:47
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.

3 participants