Skip to content
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.

Sign up for GitHub

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

Jump to bottom

SPE-93 Migrate cow_protocol models downstream of dex_trades #4341

Merged
merged 11 commits into from
Oct 2, 2023
Merged

SPE-93 Migrate cow_protocol models downstream of dex_trades #4341

merged 11 commits into from
Oct 2, 2023

Conversation

couralex6
Copy link
Contributor

@couralex6 couralex6 commented Sep 14, 2023 •
edited
Loading 

models/cow_protocol/tx_hash_labels/cow_protocol_tx_hash_labels_all.sql
models/cow_protocol/tx_hash_labels/bluechip_investment/cow_protocol_tx_hash_labels_bluechip_investment.sql
models/cow_protocol/tx_hash_labels/bluechip_investment/ethereum/cow_protocol_tx_hash_labels_bluechip_investment_ethereum.sql
models/cow_protocol/tx_hash_labels/early_investment/cow_protocol_tx_hash_labels_early_investment.sql
models/cow_protocol/tx_hash_labels/early_investment/ethereum/cow_protocol_tx_hash_labels_early_investment_ethereum.sql
models/cow_protocol/tx_hash_labels/harvest_yield/cow_protocol_tx_hash_labels_harvest_yield.sql
models/cow_protocol/tx_hash_labels/harvest_yield/ethereum/cow_protocol_tx_hash_labels_harvest_yield_ethereum.sql
models/cow_protocol/tx_hash_labels/offramp/cow_protocol_tx_hash_labels_offramp.sql
models/cow_protocol/tx_hash_labels/offramp/ethereum/cow_protocol_tx_hash_labels_offramp_ethereum.sql
models/cow_protocol/tx_hash_labels/onramp/cow_protocol_tx_hash_labels_onramp.sql
models/cow_protocol/tx_hash_labels/onramp/ethereum/cow_protocol_tx_hash_labels_onramp_ethereum.sql
models/cow_protocol/tx_hash_labels/stable_to_stable/cow_protocol_tx_hash_labels_stable_to_stable.sql
models/cow_protocol/tx_hash_labels/stable_to_stable/ethereum/cow_protocol_tx_hash_labels_stable_to_stable_ethereum.sql
models/cow_protocol/tx_hash_labels/staking_token_investment/cow_protocol_tx_hash_labels_staking_token_investment.sql
models/cow_protocol/tx_hash_labels/staking_token_investment/ethereum/cow_protocol_tx_hash_labels_staking_token_investment_ethereum.sql
models/cow_protocol/tx_hash_labels/treasury_management/cow_protocol_tx_hash_labels_treasury_management.sql
models/cow_protocol/tx_hash_labels/treasury_management/ethereum/cow_protocol_tx_hash_labels_treasury_management_ethereum.sql

@couralex6 couralex6 changed the title Migrate cow_protocol models downstream of dex_trades SPE-93 Migrate cow_protocol models downstream of dex_trades Sep 14, 2023
@Hosuke Hosuke mentioned this pull request Sep 30, 2023
Closed
@jeff-dude
Copy link
Member

this was just migrated:
#3809

after a successful prod run, should be able to refresh this PR and try again

@couralex6
Copy link
Contributor Author

@jeff-dude canceling the ci run because i have some local changes and this is gonna be a massive ci run

@jeff-dude
Copy link
Member

@jeff-dude canceling the ci run because i have some local changes and this is gonna be a massive ci run

okay, no problem, feel free to take it from here

@couralex6
Copy link
Contributor Author

Added these two models since they're upstream dependencies:

models/tokens/ethereum/tokens_ethereum_erc20_stablecoins.sql
models/tokens/fantom/tokens_fantom_erc20_stablecoins.sql

@couralex6 couralex6 added the ready-for-review this PR development is complete, please review label Oct 2, 2023
@couralex6
Copy link
Contributor Author

Good to go! @jeff-dude

@jeff-dude
Copy link
Member

Good to go! @jeff-dude

i won't be around the next few hours, so if you want to merge/monitor/be ready for possible reverts, feel free to go ahead 🤝

@jeff-dude jeff-dude added ready-for-merging and removed ready-for-review this PR development is complete, please review labels Oct 2, 2023
@couralex6 couralex6 merged commit 84ea752 into main Oct 2, 2023
@couralex6 couralex6 deleted the cow_protocol_labels_migration branch October 2, 2023 20:35
@github-actions github-actions bot locked and limited conversation to collaborators Oct 2, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jeff-dude jeff-dude jeff-dude approved these changes

Assignees
No one assigned
Labels
ready-for-merging
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@couralex6 @jeff-dude @aalan3