Skip to content

Conversation

@mz-
Copy link
Contributor

@mz- mz- commented Aug 22, 2024

we need to index events from the new PositionManager contract, more context: https://uniswapteam.slack.com/archives/C06PJT9F3HT/p1723129723983949?thread_ts=1722884945.815439&cid=C06PJT9F3HT

the primary use case to unblock LP work will be to be able to fetch the PositionConfig by tokenId. This adds a mint event handler

@mz- mz- requested a review from matteenm August 22, 2024 18:44
@mz- mz- changed the title index mint position events index transfer, subscribe, unsubscribe events Sep 26, 2024
Copy link
Collaborator

@matteenm matteenm left a comment

Choose a reason for hiding this comment

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

lgtm assuming some manual testing was done

@mz- mz- merged commit 968baef into main Sep 30, 2024
4 checks passed
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