Skip to content

Add native token price mappings and fix chain IDs - #9912

Open
pandajackson42 wants to merge 3 commits into
duneanalytics:mainfrom
pandajackson42:panda/add-prices-native-token-info-fix-chain-id
Open

Add native token price mappings and fix chain IDs#9912
pandajackson42 wants to merge 3 commits into
duneanalytics:mainfrom
pandajackson42:panda/add-prices-native-token-info-fix-chain-id

Conversation

@pandajackson42

@pandajackson42 pandajackson42 commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add Robinhood and TAC native token price mappings.
  • Update B3 chain ID to 8333, TAC chain ID to 239.
  • Fix TAC native_token_symbol & wrapped_native_token_address.
  • Add Flow into gas_evm_chains() macro.

Validation

  • checked coinpaprika id through api.coinpaprika.com/v1/coins
  • checked chain id of B3 and TAC through chainlist.org

@github-actions github-actions Bot added WIP work in progress dbt: daily covers the Daily dbt subproject dbt: tokens covers the Tokens dbt subproject labels Jul 31, 2026
@pandajackson42 pandajackson42 changed the title Fix native token price mappings and chain IDs Add native token price mappings and fix chain IDs Jul 31, 2026
@pandajackson42
pandajackson42 marked this pull request as ready for review July 31, 2026 08:45
@github-actions github-actions Bot added ready-for-review this PR development is complete, please review and removed WIP work in progress labels Jul 31, 2026
@cursor

cursor Bot commented Jul 31, 2026

Copy link
Copy Markdown

PR Summary

Medium Risk
Wrong chain IDs can mis-attribute EVM data across pipelines; new price mappings affect USD valuation for those chains but changes are limited to static reference tables.

Overview
Corrects B3 and TAC chain_id values in evms_info so they match canonical chain metadata (B3: 11010018333, TAC: 10000239); other chain fields are unchanged.

Adds Coinpaprika native-token mappings in prices_native_tokens for Robinhood (eth-ethereum) and TAC (tac-tac-protocol) so price joins can resolve those blockchains.

Reviewed by Cursor Bugbot for commit 835f5cb. Configure here.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is ON, but it could not run because the branch was deleted or merged before autofix could start.

Want higher recall? High effort reviews run extra passes and find more bugs. A team admin can switch effort levels in the Cursor dashboard.

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 835f5cb. Configure here.

Comment thread dbt_subprojects/daily_spellbook/models/evms/evms_info.sql Outdated
@github-actions github-actions Bot added the dbt: hourly covers the hourly dbt subproject label Aug 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dbt: daily covers the Daily dbt subproject dbt: hourly covers the hourly dbt subproject dbt: tokens covers the Tokens dbt subproject ready-for-review this PR development is complete, please review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant