-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Upgrade Event Models for GMX v2 Project #7195
Upgrade Event Models for GMX v2 Project #7195
Conversation
Commits updated models for the Arbitrum blockchain. The main goal is to verify that we can successfully update models before scaling the process to Avalanche. This implies that the PR will not go through all checks but must create models for the Arbitrum blockchain. Once this is done, we will manually review the tables and then repeat the process for Avalanche. |
marking blocked, until i see if you'd like to explore the incremental design on spells here as well similar to #7187 let me know what you think 🙏 |
Let's proceed as follows: We'll merge this PR since it was created to align some of the dbt models with updates to the smart contracts. After that, I will pull all changes from the main branch and add incremental updates in a second PR, where I am working on adding new dbt models and designing incremental changes. Is it fine for you? |
yep, works for me, thanks! |
we are getting a failure in CI now though, so i'll wait to see that resolved |
Now it's ready for merge! :) Sometimes, errors occur because the reference model was not updated, resulting in NULL values for columns that cannot be NULL. |
Description:
Upgrade 4 Event Models: OrderCreated, OrderUpdated, PositionFeesCollected and PositionFeesInfo