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

Feat/update interface for pool tracker #154

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

hiepnv90
Copy link
Member

Update IPoolTracker interface for accepting more params

Why did we need it?

We need to pass event logs for pool to handle properly:

  • We need to fetch pool state from node at a specified block number
  • We can use event logs to update uniswapv2 reserves instead of fetching it from node to reduce cost

Related Issue

Release Note

How Has This Been Tested?

Screenshots (if appropriate):

@hiepnv90 hiepnv90 requested a review from 0xh9n as a code owner September 21, 2023 08:16
@hiepnv90 hiepnv90 self-assigned this Sep 21, 2023
@hiepnv90 hiepnv90 requested a review from piavgh as a code owner September 21, 2023 08:16
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