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

Swapline DEX on Base #7534

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

PatelPrinci
Copy link
Contributor

Thank you for contributing to Spellbook 🪄

Please open the PR in draft and mark as ready when you want to request a review.

Description:

[...]


quick links for more information:

@github-actions github-actions bot marked this pull request as draft January 24, 2025 09:28
@github-actions github-actions bot added WIP work in progress dbt: dex covers the DEX dbt subproject dbt: hourly covers the hourly dbt subproject labels Jan 24, 2025
@github-actions github-actions bot removed the dbt: hourly covers the hourly dbt subproject label Jan 24, 2025
@Hosuke
Copy link
Collaborator

Hosuke commented Jan 24, 2025

Duped PR with #7532 ?

@PatelPrinci PatelPrinci marked this pull request as ready for review January 24, 2025 10:36
@github-actions github-actions bot added ready-for-review this PR development is complete, please review and removed WIP work in progress labels Jan 24, 2025
@PatelPrinci
Copy link
Contributor Author

PatelPrinci commented Jan 24, 2025 via email

@jeff-dude jeff-dude marked this pull request as draft January 24, 2025 15:50
@github-actions github-actions bot added WIP work in progress and removed ready-for-review this PR development is complete, please review labels Jan 24, 2025
@PatelPrinci
Copy link
Contributor Author

@Hosuke
Could you please take a look at this?
In your previous pull request comment, you suggested using the Uniswap V2-compatible model for Swapline. However, in my earlier submission, I didn’t use the Uniswap V2 model because it was causing errors due to column mismatches. Instead, I created my own dbt model to address those issues and submitted it.

This time, I’ve attempted to implement the method using the Uniswap V2 model as you recommended, but I’m encountering an error again. Could you please help me troubleshoot and resolve the issue? Your guidance would be greatly appreciated.

Copy link
Collaborator

@Hosuke Hosuke left a comment

Choose a reason for hiding this comment

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

Please verify the schema of sources used.
Try using the swapline_base.swaplinepair_evt_swap source.

blockchain = 'base',
project = 'swapline',
version = '1',
Pair_evt_Swap = source('swapline_base', 'LBPair_evt_Swap'),
Copy link
Collaborator

Choose a reason for hiding this comment

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

Sorry I could not find this source in the data explorer.

@PatelPrinci
Copy link
Contributor Author

PatelPrinci commented Jan 30, 2025 via email

@Hosuke
Copy link
Collaborator

Hosuke commented Jan 30, 2025

I see...
Maybe try trader_joe_compatible_v2_trades macro here for LBPair tables?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dbt: dex covers the DEX dbt subproject WIP work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants