Skip to content

Conversation

@alicecha
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 added WIP work in progress dbt: dex covers the DEX dbt subproject labels Dec 23, 2025
@alicecha alicecha mentioned this pull request Dec 23, 2025
@alicecha alicecha requested a review from 0xBoxer January 8, 2026 16:19
Copy link
Collaborator

@0xBoxer 0xBoxer left a comment

Choose a reason for hiding this comment

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

Would be great if you can also create a schema.yml file roughly documenting the unions we are making for each of these columns.

choice of columns looks solid to me

@alicecha alicecha force-pushed the multichain-dex-ttrades-clean branch from e0ce131 to 1146964 Compare January 9, 2026 12:07
Copy link
Collaborator

@0xBoxer 0xBoxer left a comment

Choose a reason for hiding this comment

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

What about Sui and TON dex trades, is it too hard to standardise them in here?

@alicecha
Copy link
Contributor Author

alicecha commented Jan 9, 2026

What about Sui and TON dex trades, is it too hard to standardise them in here?

I looked briefly but need more time to validate the data quality and standardise. Should be doable by dedicating ~1 day.

@alicecha alicecha requested a review from 0xBoxer January 9, 2026 15:49
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.

3 participants