Update balancer_cowswap_amm_ethereum_liquidity.sql

[viniabussafi]

Thank you for contributing to Spellbook 🪄

Update!

Please build spells in the proper subproject directory. For more information, please see the main readme, which also links to a GH discussion with the option to ask questions.

Contribution type

Please check the type of contribution this pull request is for:

• New spell(s)
• Adding to existing spell lineage
• Bug fix

Note: You can safely discard any section below which doesn't apply based on selection above

---

Quick update to fix the protocol_liquidity_eth and pool_liquidity_eth fields on the balancer liquidity models as the former syntax (WHERE symbol = 'ETH') was causing undesired results, likely on either one of these results:

2024-09-03 08:47	solana	0x0b931265156395f7491826c94b706e34717e5a9c60e6f668ff9301bbc347b8c1	8	ETH	0.00000884
2024-09-03 08:47	ethereum	0x0d505c03d30e65f6e9b4ef88855a47a89e4b7676	18	ETH	0.00000884

Side note: I've seen that some nexusmutual spells also use this syntax, so tagging @tomfutago here for visibility

[dune-eng]

Workflow run id 10679654030 approved.

[dune-eng]

Workflow run id 10679653842 approved.

[dune-eng]

Workflow run id 10679665656 approved.

[dune-eng]

Workflow run id 10679665797 approved.

[dune-eng]

Workflow run id 10679678556 approved.

[dune-eng]

Workflow run id 10679678925 approved.

[dune-eng]

Workflow run id 10679703075 approved.

[dune-eng]

Workflow run id 10679703389 approved.

[dune-eng]

Workflow run id 10679791351 approved.

[dune-eng]

Workflow run id 10679791310 approved.

[dune-eng]

Workflow run id 10679808690 approved.

[dune-eng]

Workflow run id 10679808893 approved.

[dune-eng]

Workflow run id 10679818865 approved.

[dune-eng]

Workflow run id 10679819110 approved.

[tomfutago]

Side note: I've seen that some nexusmutual spells also use this syntax, so tagging @tomfutago here for visibility

hey @viniabussafi thanks for bringing this up! nexusmutual spells are using this type of syntax (hence I think they're fine for the time being):

  where symbol = 'ETH'
    and blockchain is null
    and contract_address is null

there's an open issue tho: #6577 with suggestions how price feed for native tokens might change, so worth following and I agree with point raised there that using WETH for ETH prices is not the best approach (risk of depeg events).

[jeff-dude]

yes, plz stay tuned for enhanced approach to pricing ideally released this month 🫡