-
Notifications
You must be signed in to change notification settings - Fork 1.4k
CUR2-1212 multichain dex trades clean #9155
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
base: main
Are you sure you want to change the base?
Conversation
0xBoxer
left a comment
There was a problem hiding this 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
e0ce131 to
1146964
Compare
0xBoxer
left a comment
There was a problem hiding this 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?
I looked briefly but need more time to validate the data quality and standardise. Should be doable by dedicating ~1 day. |
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: