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

Updated IBC Denoms and channels #5904

Closed
wants to merge 50 commits into from

Conversation

FlatWhiteWithSugar
Copy link
Contributor

Updated IBC Denoms and channels

	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
Updated changes
added polkachu
added additional IBC connections for kopi
Updated to use correct client id
Updated details
updated asset list with atom, luna and osmosis
added arbitrage tokens
removed, socials from tokens, added in missing traces, updated injective display name, and fixed issue with capital letter on chain name
Updated asUSDT.inj to correct Trace & fixed symbol for Injective
Updated the Trace
Remove traces for kopi generated tokens
Updated the as and c denoms to upper case
Updated IBC to  "type_asset": "ics20",
remove versions and updated the image path
Update osmosis to lower case and fixed issue with luna
Updated the image path
@@ -213,11 +213,11 @@
"counterparty": {
"chain_name": "noble",
"base_denom": "uusdc",
"channel_id": "channel-109"
"channel_id": "channel-122"
Copy link
Collaborator

Choose a reason for hiding this comment

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

This channel is incorrect. The IBC connection between noble and kopi is already registered. That registration must be updated first, and that may only be changed with governance agreement or proof that there are no assets abandoned on the counterparty chains (both ways).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

noble-assets/noble#477 we have reached out to noble to get the client updated

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have reverted the changes for noble

updated kopi to noble channels
@JeremyParish69
Copy link
Collaborator

It looks like a lot of these assets are referring to IBC connections that aren't registered, causes checks to fail. Please only update for registered IBC connections

@JeremyParish69
Copy link
Collaborator

It looks like there is a small amount of XKP transferred over to Osmosis. Small testing amounts may abandoned if the remaining evidence is solid
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants