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.

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

create tokens_{chain}.info #6970

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Conversation

hildobby
Copy link
Collaborator

@hildobby hildobby commented Oct 18, 2024

This can be considered as an improved version to tokens_{chain}.erc20 which imo is poorly named, lacks native tokens and a lot of other columns.

A crosschain spell doesn't make sense for now I will eventually work on a spell that maps tokens across different chains (ie all USDC addresses across chains). Once that's in place, we can combine it with the results from this spell to get a great crosschain spell.

For downstream usage, this will come in handy to rank tokens based on heuristics to filter out spam tokens from the rest for example. That will in turn be useful for spells like: attacks.address_poisonning

@hildobby hildobby added the WIP work in progress label Oct 18, 2024
@0xBoxer
Copy link
Collaborator

0xBoxer commented Oct 23, 2024

Due to a dbt version change, you need to merge main into your branch before the Github action runner runs successfully again.

@0xRobin 0xRobin marked this pull request as draft October 28, 2024 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WIP work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants