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

[spooky-lp-sonic] Added spooky-lp-sonic strategy and updated index.ts #1677

Merged
merged 6 commits into from
Jan 23, 2025

Conversation

0xaaiden
Copy link
Contributor

This pull request introduces a new strategy called spooky-lp-sonic for calculating liquidity positions in $BOO pools. The changes include adding the new strategy to the list of strategies, creating a README file for documentation, providing example queries, and implementing the strategy logic.

New Strategy Implementation:

  • Strategy Addition:

    • Added spooky-lp-sonic to the list of strategies in src/strategies/index.ts.
  • Documentation:

    • Created src/strategies/spooky-lp-sonic/README.md to document the new strategy, including an overview of the pools queried and the method used to calculate net liquidity.
  • Example Queries:

    • Added src/strategies/spooky-lp-sonic/examples.json to provide example queries for the new strategy.
  • Strategy Logic:

    • Implemented the spooky-lp-sonic strategy in src/strategies/spooky-lp-sonic/index.ts, including functions to query subgraph data, handle pagination, and calculate net liquidity positions.

@0xaaiden
Copy link
Contributor Author

@ChaituVR

@ChaituVR ChaituVR changed the title Added spooky-lp-sonic strategy and updated index.ts [spooky-lp-sonic] Added spooky-lp-sonic strategy and updated index.ts Jan 22, 2025
@0xaaiden
Copy link
Contributor Author

@ChaituVR it should be fixed

src/strategies/spooky-lp-sonic/index.ts Outdated Show resolved Hide resolved
src/strategies/spooky-lp-sonic/index.ts Outdated Show resolved Hide resolved
@0xaaiden
Copy link
Contributor Author

@ChaituVR fixed again!

src/strategies/index.ts Outdated Show resolved Hide resolved
@ChaituVR ChaituVR merged commit e4d0c9a into snapshot-labs:master Jan 23, 2025
3 checks passed
@0xaaiden
Copy link
Contributor Author

@ChaituVR how long does it take for the strategy to show up on snapshot

Copy link
Contributor Author

@0xaaiden 0xaaiden left a comment

Choose a reason for hiding this comment

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

@ChaituVR can you reopen the PR, there is an issue with the network id, it should be 146 for sonic instead of ethereum @BounebRayan

@ChaituVR
Copy link
Member

@0xaaiden Feel free to create a new PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants