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

Add Pundi X Chain #321

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Conversation

zakir-code
Copy link
Contributor

No description provided.

Copy link

vercel bot commented Dec 13, 2023

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

@HeesungB
Copy link
Collaborator

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

@zakir-code
Copy link
Contributor Author

@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"
  }
}

@malm0d
Copy link

malm0d commented Mar 13, 2024

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?

@HeesungB
Copy link
Collaborator

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

@zakir-code zakir-code closed this Mar 28, 2024
@zakir-code
Copy link
Contributor Author

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

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.

4 participants