Updating pokt testnet chain json#1439
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: caecc8f1c6
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| "bech32PrefixConsAddr": "poktvalcons", | ||
| "bech32PrefixConsPub": "poktvalconspub" | ||
| }, | ||
| "walletUrlForStaking": "https://wallet.keplr.app/chains/pocket-network?modal=staking&chain=pocket&step_id=2", |
There was a problem hiding this comment.
Point staking URL to the testnet chain id
walletUrlForStaking was added with chain=pocket, which targets Pocket mainnet, while this config is for chainId pocket-lego-testnet; as a result, users clicking Keplr’s staking button from this testnet entry will be redirected to the wrong network context. In this repo, the same URL is used by cosmos/pocket.json (mainnet), so reusing it here appears to be a copy-over mismatch rather than a testnet-specific staking route.
Useful? React with 👍 / 👎.
Changes
Checklist
yarn validate <your-config-file>locally, and it passed without any errors.