Update meteora_v2_solana_base_trades.sql

[DavidbigC]

Thank you for contributing to Spellbook 🪄

Please open the PR in draft and mark as ready when you want to request a review.

Description:

[...]

---

quick links for more information:

• README.md
• spellbook docs
• CONTRIBUTING.md

[DavidbigC]

I made the changes because Meteora doesnt really follow the kind of inner_instruction_index + 1 + 2 format.
for most trades, it does, until it does not.
e.g.,
51WeczDXGARQdSeRiR4g5iNSw95iGZvGcrccbsjW3jj1GY2SSP4pUSh37Eu2UbToSt8hBrRac1fRvEaZoAgVjRfU
the first transfer send 0.00473 to the same account, which was logged as the sold value in the original table

I am joining tables by to_owner & from_owner.
since its a dex, they will always point to the lp - lbpair

[Copilot]

Copilot wasn't able to review any files in this pull request.

Files not reviewed (1)

• dbt_subprojects/solana/models/_sector/dex/meteora/meteora_v2_solana_base_trades.sql: Language not supported

[0xBoxer]

closing this in favour of #8094 , which was easier to work on for me.

thanks for the work @DavidbigC, could verify and fix the issue in the #8094 branch by adding an additional join condition for the transfers join. We are still finishing up some work on tokens.transfers so this will take another couple days to be merged into prod