Skip to content

CUR2-263 taiko to base trades #8502

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 34 commits into from
Jul 25, 2025
Merged

CUR2-263 taiko to base trades #8502

merged 34 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 18, 2025 15:27
@github-actions github-actions bot marked this pull request as draft July 18, 2025 15:27
@github-actions github-actions bot added WIP work in progress dbt: dex covers the DEX dbt subproject labels Jul 18, 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 implements DEX (Decentralized Exchange) trade tracking for the Taiko blockchain by adding base trade models for four platforms: Oku, Unagi V2/V3, and Izumi. The implementation follows the standard Spellbook pattern for DEX integrations.

  • Adds source definitions for DEX event tables from four platforms on Taiko
  • Creates individual base trade models for each platform using existing Uniswap-compatible macros
  • Implements a unified aggregation model that combines all platform trades
  • Adds comprehensive test coverage with seed data validation

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
sources/_sector/dex/trades/taiko/_sources.yml Defines event table sources for Oku, Unagi V2/V3, and Izumi DEX platforms
dbt_subprojects/dex/seeds/trades/_schema.yml Adds seed data schemas for testing each platform's base trades
dbt_subprojects/dex/models/trades/taiko/platforms/*.sql Individual platform models that transform raw events into standardized trade data
dbt_subprojects/dex/models/trades/taiko/dex_taiko_base_trades.sql Unified model aggregating all platform trades with transaction metadata
dbt_subprojects/dex/models/trades/taiko/_schema.yml Defines model metadata, tests, and documentation for all Taiko DEX models
Comments suppressed due to low confidence (1)

dbt_subprojects/dex/models/trades/taiko/platforms/oku_v3_taiko_base_trades.sql:3

  • The schema name 'oku_v3_taiko' is inconsistent with the project name. Based on the source definition, this should be 'oku_taiko' to match the source name 'oku_taiko'.
        schema = 'oku_v3_taiko',

@krishgka krishgka changed the title test output from base trades CUR2-263 taiko to base trades Jul 18, 2025
@krishgka krishgka marked this pull request as ready for review July 22, 2025 20:33
@github-actions github-actions bot added ready-for-review this PR development is complete, please review and removed WIP work in progress labels Jul 22, 2025
@jeff-dude jeff-dude self-assigned this Jul 22, 2025
@jeff-dude jeff-dude added in review Assignee is currently reviewing the PR and removed ready-for-review this PR development is complete, please review labels Jul 22, 2025
@krishgka krishgka added ready-for-review this PR development is complete, please review and removed in review Assignee is currently reviewing the PR labels Jul 23, 2025
@jeff-dude jeff-dude added ready-for-merging and removed ready-for-review this PR development is complete, please review labels Jul 23, 2025
Copy link
Member

@jeff-dude jeff-dude left a comment

Choose a reason for hiding this comment

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

missing the amount_usd check

@jeff-dude jeff-dude marked this pull request as draft July 25, 2025 16:26
@github-actions github-actions bot added the WIP work in progress label Jul 25, 2025
@jeff-dude jeff-dude marked this pull request as ready for review July 25, 2025 18:11
@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 7e1c6c9 into main Jul 25, 2025
4 of 5 checks passed
@jeff-dude jeff-dude deleted the taiko_into_trades branch July 25, 2025 19:39
@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