Skip to content

Add Xyra Chain (xyra-2)#7577

Merged
CryptoAssassin1 merged 17 commits intocosmos:masterfrom
rizkimaryanto7-creator:master
Mar 24, 2026
Merged

Add Xyra Chain (xyra-2)#7577
CryptoAssassin1 merged 17 commits intocosmos:masterfrom
rizkimaryanto7-creator:master

Conversation

@rizkimaryanto7-creator
Copy link
Copy Markdown
Contributor

Adding Xyra Chain configuration.

@rizkimaryanto7-creator
Copy link
Copy Markdown
Contributor Author

Hi maintainers, I've updated the repository to public and verified all links. Ready for the automated checks. Thank you!

@rizkimaryanto7-creator
Copy link
Copy Markdown
Contributor Author

Hello! I've updated the assetlist.json for Xyra Chain. I fixed the nested logo_URIs object and standardized the format. All checks are now passing (green). Please let me know if any further changes are needed. Thanks!

@rizkimaryanto7-creator
Copy link
Copy Markdown
Contributor Author

Hi @maintainers, I have fixed my endpoints and verified them. Although the CI shows a global failure due to other chains' issues, Xyra-Chain endpoints are now passing correctly (Status 200). You can check the logs for Xyra. Ready for review and merge. Thank you!

@rizkimaryanto7-creator
Copy link
Copy Markdown
Contributor Author

Hi, I have fixed all the validation issues, including the type_asset field and logo dimensions. The PR is now passing all checks. Ready for review and merge. Thank you!

xyra/chain.json Outdated
"recommended_version": "v2.0.0",
"compatible_versions": ["v2.0.0"],
"genesis": {
"genesis_url": "https://raw.githubusercontent.com/rizkimaryanto7-creator/xyra-chain/main/genesis.json"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Returns 404

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Hi @CryptoAssassin1, the issue was a branch name mismatch. I've updated the genesis_url to the correct branch (master). It should be working now. Thanks for the catch!

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

it's not a correct SVG file. A valid SVG element cannot have two viewBox attributes.

"display": "XYRIUM",
"name": "XYRIUM",
"symbol": "XYR",
"logo_URIs": {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Registry standard requires both logo_URIs and images. Please add images

@CryptoAssassin1 CryptoAssassin1 self-assigned this Mar 13, 2026
@rizkimaryanto7-creator
Copy link
Copy Markdown
Contributor Author

Hi @CryptoAssassin1, I've addressed your feedback in the latest commit:

  1. Fixed the SVG file by removing the duplicate viewBox attribute.

  2. Added the required images array to assetlist.json to comply with the registry standard.

Thank you for the thorough review! Everything should be in order now.

@rizkimaryanto7-creator
Copy link
Copy Markdown
Contributor Author

Hi @CryptoAssassin1, I have updated assetlist.json to include both logo_URIs and the images array as requested. I also cleaned up the JSON syntax. Ready for review. Thanks!

@CryptoAssassin1
Copy link
Copy Markdown
Member

@rizkimaryanto7-creator 1. recommended_version v2.0.0 doesn't exist

  1. Missing images and logo_URIs in chain.json
    Please add the logo_URIs and images fields referencing xyra/images/xrc.png and xrc.svg.

  2. Missing peers
    Please add at least one seed or persistent peer.

  3. Explorer unreachable
    https://explorer.xyra-mainnet.org is not responding.

@rizkimaryanto7-creator
Copy link
Copy Markdown
Contributor Author

Hi @CryptoAssassin1,

Thank you for your detailed feedback. I have updated the chain.json and resolved the technical issues as requested:

Codebase & Versioning: Corrected the recommended_version to v1.0.0 and ensured all links point to the stable release.

Peers: Added a persistent peer (72.62.121.184:26656) to ensure network connectivity.

Metadata & Images: Included the missing logo_URIs and images fields with the correct GitHub raw paths for both PNG and SVG formats.

Explorer Access: Resolved the connectivity issue. The explorer is now fully operational and can be reached at https://explorer.xyra-mainnet.org/.

The configuration has been validated against the registry schema. Please let me know if any further adjustments are needed. Looking forward to your review.

@CryptoAssassin1
Copy link
Copy Markdown
Member

Hi @CryptoAssassin1,

Thank you for your detailed feedback. I have updated the chain.json and resolved the technical issues as requested:

Codebase & Versioning: Corrected the recommended_version to v1.0.0 and ensured all links point to the stable release.

Peers: Added a persistent peer (72.62.121.184:26656) to ensure network connectivity.

Metadata & Images: Included the missing logo_URIs and images fields with the correct GitHub raw paths for both PNG and SVG formats.

Explorer Access: Resolved the connectivity issue. The explorer is now fully operational and can be reached at https://explorer.xyra-mainnet.org/.

The configuration has been validated against the registry schema. Please let me know if any further adjustments are needed. Looking forward to your review.

  1. recommended_version: "v2.0.0" doesn't exist only v1.0.0 is tagged.
  2. please add ,missing logo_URIs and images in chain.json
  3. Missing peers please add at least one seed or persistent peer with a valid 40-character hex node ID.
  4. chain.json formatting, there are blank lines between every field which is non-standard. Please remove them so the JSON is compact and consistent with the rest of the registry. For reference, see any existing chain like cosmoshub/chain.json.

@rizkimaryanto7-creator
Copy link
Copy Markdown
Contributor Author

I've updated the chain.json to v1.0.0, added the persistent peer with Node ID, included logo URIs, and formatted the JSON to be compact. Ready for another review!

@CryptoAssassin1 CryptoAssassin1 merged commit 424eabf into cosmos:master Mar 24, 2026
4 checks passed
@rizkimaryanto7-creator
Copy link
Copy Markdown
Contributor Author

Thanks a lot @CryptoAssassin1 for the review and for merging the Xyra Chain registry! Appreciate your help in getting us through the validation process. 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants