-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Update/create stablecoin token sets. #6186
Changes from all commits
813ff51
d9d17e7
46ab050
8ed69e2
c16c01a
f129a15
fbc5ed1
d36fab7
35d83bc
b00ce05
ae314b9
4bbfd83
1262140
2c49297
4c47ed7
250f33c
4112c9f
788d48c
9ce6f7a
a546fda
6e35bf6
8b5aa54
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. i'm not sure we should delete these, as they may have downstream dependencies or get used by a few users/dashboards on dune? |
This file was deleted.
This file was deleted.
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. similar to all other chains, i would check these already exist, but since this is a net new file it's worth a close look. we can remove the file altogether if they exist |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,19 @@ | ||
{{ | ||
config( | ||
schema = 'tokens_linea' | ||
, alias = 'erc20' | ||
, tags = ['static'] | ||
, materialized = 'table' | ||
) | ||
}} | ||
|
||
SELECT contract_address, symbol, decimals | ||
FROM ( | ||
VALUES | ||
(0x176211869ca2b568f2a7d4ee941e073a821ee1ff, 'USDC', 6) --bridged | ||
, (0xb79dd08ea68a908a97220c76d19a6aa9cbde4376, 'USD+', 6) | ||
, (0xA219439258ca9da29E9Cc4cE5596924745e12B93, 'USDT', 6) | ||
, (0x4af15ec2a0bd43db75dd04e62faa3b8ef36b00d5, 'DAI', 18) | ||
, (0xd2bc272EA0154A93bf00191c8a1DB23E67643EC5, 'USDP', 18) | ||
, (0xDD3B8084AF79B9BaE3D1b668c0De08CCC2C9429A, 'MIM', 18) | ||
) AS temp_table (contract_address, symbol, decimals) |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,21 @@ | ||
version: 2 | ||
|
||
models: | ||
- name: tokens_linea_erc20 | ||
meta: | ||
blockchain: linea | ||
sector: tokens | ||
project: erc20 | ||
contributors: thetroyharris | ||
config: | ||
tags: [ 'table', 'erc20', 'linea' ] | ||
description: "ERC20 Token Addresses, Symbols and Decimals" | ||
columns: | ||
- name: contract_address | ||
description: "ERC20 token contract address" | ||
tests: | ||
- unique | ||
- name: symbol | ||
description: "ERC20 token symbol" | ||
- name: decimals | ||
description: "Number of decimals, refers to how divisible an ERC20 token can be" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i'm still a bit confused on this approach here. these stablecoins already exist in
tokens.erc20
:keep in mind that erc20 token metadata now combines an automated source + these static files to handle outlier, long tail tokens missing. i would validate all of these already exist in the table, then remove from this PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wow did I ever jump the gun on this. This hurts my soul.