SPE-4 Migrate & rework dex.sandwiches

[hildobby]

Thank you for contributing to Spellbook!

Please refer to the top of the readme in the root of Spellbook to learn how to contribute to Spellbook on DuneSQL.

[hildobby]

Huh "Table 'hive.dex.trades' does not exist"? What did I do wrong here @jeff-dude?

[Hosuke]

Huh "Table 'hive.dex.trades' does not exist"? What did I do wrong here @jeff-dude?

This means dex.trades is not migrated yet(The spell does not have the dunesql tag).

We may use dex.trades_migration_beta instead for current testing.

[hildobby]

trades_migration_beta

Alright thanks, I'll try out using dex.trades_migration_beta instead then

[jeff-dude]

trades_migration_beta

Alright thanks, I'll try out using dex.trades_migration_beta instead then

well, good news is dex.trades is now live on dunesql 🙌

less good news, you'll have to change code back 🤷‍♂️

[hildobby]

trades_migration_beta

Alright thanks, I'll try out using dex.trades_migration_beta instead then

well, good news is dex.trades is now live on dunesql 🙌

less good news, you'll have to change code back 🤷‍♂️

Damn that was quick wtf, I swear the migration started barely a week ago

[0xRobin]

@hildobby I was actually able to run this successfully locally

@couralex6 curious what model you were able to run locally, as there seems to be still some issues in the macro that should make these models fail to run. 🤔

[couralex6]

@hildobby I was actually able to run this successfully locally

@couralex6 curious what model you were able to run locally, as there seems to be still some issues in the macro that should make these models fail to run. 🤔

Oops you're right I was running on old version of the branch

[Hosuke]

Currently only sandwiches records on ethereum is available:
https://dune.com/queries/3056258

[jeff-dude]

beautiful PR here! love to see the use of macros becoming the norm, it makes everyone's lives easier.

thanks again for all the help in migration 🤝