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

chore(v3 amm): [sc-2378] Upgrade NFTX AMM Subgraph to the new Uniswap Subgraph #21

Merged

Conversation

jackmellis
Copy link
Collaborator

this new uniswap subgraph works much better for nftx.js's needs than the messari one
this commit updates the subgraph urls, schemas, and transformations to work with the new subgraph

all api-level types (liquidityPools/posittions/vaults) maintain the same structure as before. however, the underlying subgraph types and urls have all changed and are considered breaking.
this only directly affects internal methods (api/indexer/@nftx/core) it has not been marked as breaking, the public API remains the same
@jackmellis jackmellis marked this pull request as ready for review April 18, 2024 13:52
@jackmellis jackmellis merged commit a9912b3 into main Apr 18, 2024
1 check passed
@jackmellis jackmellis deleted the chore/sc-2378/upgrade-nftx-amm-subgraph-to-the-new-uniswap branch April 18, 2024 18:02
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