-
Notifications
You must be signed in to change notification settings - Fork 1.4k
zereox - add monad #9136
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?
zereox - add monad #9136
Conversation
PR SummaryIntroduce Monad support for 0x by adding v2 trades and settler addresses, wiring them into aggregated fills/trades, and updating schemas/exposures.
Written by Cursor Bugbot for commit 6c17ccb. Configure here. |
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.
Comment @cursor review or bugbot run to trigger another review on this PR
| @@ -0,0 +1,133 @@ | |||
| version: 2 | |||
|
|
|||
| monadls: | |||
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.
Bug: Typo: monadls instead of models breaks schema
The schema file uses monadls: as the top-level key instead of models:. This is clearly a typo that will cause dbt to completely ignore the model definitions, data tests, and column descriptions for both zeroex_v2_monad_trades and zeroex_monad_settler_addresses. All other blockchain schema files in the zeroex project correctly use models: as the key.
Description: