-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Terra: add autogenerated assetlist and images #343
Terra: add autogenerated assetlist and images #343
Conversation
script from https://github.com/zhongfu/chain-registry-scripts/tree/master/terra cw20 token info + images from https://github.com/terra-money/assets/blob/master/cw20/tokens.js (cosmos#105) native tokens from Terra FCD, https://fcd.terra.dev/cosmos/bank/v1beta1/denoms_metadata native token images from https://github.com/terra-money/assets/tree/master/icon
- use mainnet address for WHALE token - update name to match terra cw20 list
I don't think we should be changing symbols of tokens, which makes me think we shouldn't have (delisted) as part of the symbol, image name, or denom, but that it'd be appropriate fine for the 'Name' field. |
I've got some ideas:
|
Will you be doing that first idea (moving the 'delisted' over to the Name field) in this PR? or you prefer I do that manually after? |
alright, i've gone ahead and done that (+ fixed the PAXG image, and verified that all of the images are actually images) |
Changes:
Other things to note:
script from https://github.com/zhongfu/chain-registry-scripts/tree/master/terra
cw20 token info + images from https://github.com/terra-money/assets/blob/master/cw20/tokens.js (#105)
native tokens from Terra FCD, https://fcd.terra.dev/cosmos/bank/v1beta1/denoms_metadata
native token images from https://github.com/terra-money/assets/tree/master/icon