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

Update nUSDC, nUSDT #5800

Draft
wants to merge 17 commits into
base: master
Choose a base branch
from
Draft

Conversation

guardianofthesea
Copy link
Contributor

Add factory wrapped nUSDC, nUSDT tokens to injective list
update cw20 "type_asset" for cw20 nUSDT, nUSDC and nSOL
Add ics20 nUSDC, and nUSDT to pryzm list

"symbol": "nUSDC",
"traces": [
{
"type": "Factory-Wrap",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Factory-Wrap is not a defined type. Please use one of the defined types, such as "wrapped"

@@ -737,7 +737,7 @@
}
}
],
"type_asset": "sdk.coin"
"type_asset": "cw20"
Copy link
Collaborator

Choose a reason for hiding this comment

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

This asset doesn't have the cw20: prefix on the base denom, so it cannot be a cw20 (unless it was originally registered incorrectly, and the base denom actually should have the cw20: prefix on it).

"chain_name": "injective",
"base_denom": "inj1cy9hes20vww2yr6crvs75gxy5hpycya2hmjg9s"
},
"provider": "CW20-Wrapper"
Copy link
Collaborator

Choose a reason for hiding this comment

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

For chain-native wrapping mechanisms, we just use the chain's name as the provider (so, in this case, "Injective")

}
}
],
},
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please deal with the extra or forgotten commas to make the JSON Schema valid

@JeremyParish69 JeremyParish69 marked this pull request as draft January 14, 2025 22:23
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