-
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
add kopi mainnet #5625
base: master
Are you sure you want to change the base?
add kopi mainnet #5625
Conversation
new file: kopi/chain.json new file: kopi/images/kopi-logo.svg
Updated kopi details
updated to remove png and correct svg name
added additional explorers
removed coingecko id
added more seed and peers
updated kusd description
minor changes
minor changes
updated more details
added images and additional tokens
updated to correct image
updated text;
Updated IBC
"base": "ucusdc", | ||
"name": "cUSDC", | ||
"display": "cUSDC", | ||
"symbol": "cUSDC", |
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.
Please include the traces
describing how it originates from USDC
"base": "ukusd", | ||
"name": "kUSD", | ||
"display": "kUSD", | ||
"symbol": "kUSD", |
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.
with a lower-case k, please include the traces
of how it derives from USD
"base": "uckusd", | ||
"name": "ckUSD", | ||
"display": "ckUSD", | ||
"symbol": "ckUSD", |
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.
with a lower-case ck, please include the traces of how it derives from USD
"chain_id": "luwak-1", | ||
"daemon_name": "kopid", | ||
"node_home": "$HOME/.kopid", | ||
"bech32_prefix": "kopi", |
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.
Please register this prefix: https://github.com/satoshilabs/slips/blob/master/slip-0173.md
Updated changes
added polkachu
added additional IBC connections for kopi
kopi/assetlist.json
Outdated
@@ -0,0 +1,282 @@ | |||
{ | |||
"$schema": "../assetlist.schema.json", | |||
"chain_name": "Kopi", |
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.
the chain_name cannot have uppercase letters
"logo_URIs": { | ||
"svg": "https://raw.githubusercontent.com/cosmos/chain-registry/master/kopi/images/inj.svg" | ||
}, | ||
"images": [ |
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.
This is a repeat image. Please use the original Injective logo
"description": "injective from injective Chain", | ||
"denom_units": [ | ||
{ | ||
"denom": "ibc/DE63D8AC34B752FB7D4CAA7594145EDE1C9FC256AC6D4043D0F12310EB8FC255", |
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.
Please define the traces
(how INJ got from Injective to here)
kopi/assetlist.json
Outdated
"base": "ibc/DE63D8AC34B752FB7D4CAA7594145EDE1C9FC256AC6D4043D0F12310EB8FC255", | ||
"name": "INJ", | ||
"display": "Injective", | ||
"symbol": "Injective", |
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 believe these values are backwards. The symbol should be "INJ" and the name should be "Injective"
"description": "USDT from injective Chain", | ||
"denom_units": [ | ||
{ | ||
"denom": "ibc/D8A36AE90F20FE4843A8D249B1BCF0CCDDE35C4B605C8DED57BED20C639162D0", |
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.
Please define the traces
(how USDT got from Injective to here)
"description": "USDC from Noble Chain", | ||
"denom_units": [ | ||
{ | ||
"denom": "ibc/8E27BA2D5493AF5636760E354E46004562C46AB7EC0CC4C1CA14E9E20E2545B5", |
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.
Please define the traces
(how USDC got from Noble to here)
"base": "ucusdtinj", | ||
"name": "cUSDT.inj", | ||
"display": "cUSDT.inj", | ||
"symbol": "cUSDT.inj", |
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.
Please define the traces
(how this derives from USDT.inj (from: ibc/D8A36AE90F20FE4843A8D249B1BCF0CCDDE35C4B605C8DED57BED20C639162D0))
kopi/assetlist.json
Outdated
}, | ||
"images": [ | ||
{ | ||
"svg": "https://raw.githubusercontent.com/cosmos/chain-registry/master/kopi/images/usdc.svg" |
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.
This is a repeat image. Please use the original USDC logo
Updated to use correct client id
Updated details
No description provided.