keplr-chain-registry icon indicating copy to clipboard operation
keplr-chain-registry copied to clipboard

Add Pundi X Chain

Open zakir-web3 opened this issue 2 years ago • 6 comments

zakir-web3 avatar Dec 13 '23 11:12 zakir-web3

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
keplr-chain-registry ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 18, 2024 7:37am

vercel[bot] avatar Dec 13 '23 11:12 vercel[bot]

@zakir-code Hello. Could you remove walletUrlForStaking field? and You can not use ibc currency in currencies field. Could you replace currency?

HeesungB avatar Dec 14 '23 00:12 HeesungB

@HeesungB Hi. I have deleted walletUrlForStaking field.

Sorry, why can't we use ibc token? The bond denom currently used on the pundix chain is the PUNDIX token and mint denom is the PURSE token. https://px-rest.pundix.com/cosmos/bank/v1beta1/denoms_metadata

{
  "metadatas": [
    {
      "description": "PURSE Token",
      "denom_units": [
        {
          "denom": "bsc0x29a63f4b209c29b4dc47f06ffa896f32667dad2c",
          "exponent": 0,
          "aliases": [ ]
        },
        {
          "denom": "PURSE",
          "exponent": 18,
          "aliases": [ ]
        }
      ],
      "base": "bsc0x29a63f4b209c29b4dc47f06ffa896f32667dad2c",
      "display": "PURSE",
      "name": "PURSE Token",
      "symbol": "PURSE"
    },
    {
      "description": "Pundi X Token",
      "denom_units": [
        {
          "denom": "ibc/55367b7b6572631b78a93c66ef9fdfce87cde372cc4ed7848da78c1eb1dcdd78",
          "exponent": 0,
          "aliases": [ ]
        },
        {
          "denom": "PUNDIX",
          "exponent": 18,
          "aliases": [ ]
        }
      ],
      "base": "ibc/55367b7b6572631b78a93c66ef9fdfce87cde372cc4ed7848da78c1eb1dcdd78",
      "display": "PUNDIX",
      "name": "Pundi X Token",
      "symbol": "PUNDIX"
    }
  ],
  "pagination": {
    "next_key": null,
    "total": "2"
  }
}

zakir-web3 avatar Dec 14 '23 02:12 zakir-web3

Hi @HeesungB, we'd like to follow up on this. Is it not possible to use ibc currency in the currencies field?

Should we submit a new PR in this case?

malm0d avatar Mar 13 '24 09:03 malm0d

@zakir-code @malm0d In its current structure, CURRENCIES cannot contain IBC tokens.

HeesungB avatar Mar 14 '24 03:03 HeesungB

@HeesungB Is this possible if ibc token is removed from currencies?

zakir-web3 avatar Apr 18 '24 07:04 zakir-web3