Skip to content

CUR2-299 superseed to dex trades #8515

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

Merged
merged 21 commits into from
Jul 25, 2025
Merged

CUR2-299 superseed to dex trades #8515

merged 21 commits into from
Jul 25, 2025

Conversation

krishgka
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:

@krishgka krishgka requested review from jeff-dude and Copilot July 22, 2025 02:55
@github-actions github-actions bot marked this pull request as draft July 22, 2025 02:55
@github-actions github-actions bot added WIP work in progress dbt: dex covers the DEX dbt subproject labels Jul 22, 2025
@krishgka krishgka changed the title superseed to dex trades CUR2-299 superseed to dex trades Jul 22, 2025
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for DEX trades on the superseed blockchain, integrating two DEX protocols: BulletX (v2 and v3) and Velodrome. The implementation follows the established pattern for adding new blockchain support to the spellbook project.

  • Adds superseed blockchain support with BulletX v2/v3 and Velodrome DEX integration
  • Creates base trade models using existing uniswap-compatible macros
  • Integrates superseed trades into the main dex_base_trades aggregation

Reviewed Changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
sources/_sector/dex/trades/superseed/_sources.yml Defines source tables for BulletX and Velodrome on superseed
dbt_subprojects/dex/seeds/trades/*.csv Creates empty seed files for trade validation tests
dbt_subprojects/dex/seeds/trades/_schema.yml Adds schema definitions for the new seed files
dbt_subprojects/dex/models/trades/superseed/platforms/*.sql Implements base trade models for each DEX protocol
dbt_subprojects/dex/models/trades/superseed/dex_superseed_base_trades.sql Aggregates all superseed DEX trades
dbt_subprojects/dex/models/trades/superseed/_schema.yml Defines model schemas and tests
dbt_subprojects/dex/models/trades/dex_base_trades.sql Integrates superseed into main trades aggregation
dbt_subprojects/dex/models/dex_info.sql Adds project information for new DEX protocols

@krishgka krishgka added the ready-for-review this PR development is complete, please review label Jul 23, 2025
@krishgka krishgka removed the WIP work in progress label Jul 25, 2025
@krishgka krishgka marked this pull request as ready for review July 25, 2025 14:34
@jeff-dude jeff-dude marked this pull request as draft July 25, 2025 16:21
@jeff-dude jeff-dude self-assigned this Jul 25, 2025
@github-actions github-actions bot added WIP work in progress and removed ready-for-review this PR development is complete, please review labels Jul 25, 2025
@jeff-dude jeff-dude marked this pull request as ready for review July 25, 2025 18:55
@github-actions github-actions bot added ready-for-review this PR development is complete, please review and removed WIP work in progress labels Jul 25, 2025
@jeff-dude jeff-dude added ready-for-merging and removed ready-for-review this PR development is complete, please review labels Jul 25, 2025
@jeff-dude jeff-dude merged commit 5080091 into main Jul 25, 2025
2 of 3 checks passed
@jeff-dude jeff-dude deleted the superseed_dex_trades branch July 25, 2025 19:42
@github-actions github-actions bot locked and limited conversation to collaborators Jul 25, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dbt: dex covers the DEX dbt subproject ready-for-merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants