-
Notifications
You must be signed in to change notification settings - Fork 167
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
feat: add zaar #246
feat: add zaar #246
Conversation
WalkthroughThis pull request introduces configuration updates for multiple testnets, including Culinaris, Initia, Yominet, and Zaar. The changes primarily involve adding new metadata fields, updating asset list URLs, and introducing a new testnet configuration for Zaar. The modifications focus on enhancing network configuration, interoperability, and asset representation across different testnets. Changes
Sequence DiagramsequenceDiagram
participant Initia as Initia Testnet
participant Zaar as Zaar Testnet
Initia->>Zaar: Establish IBC Channel (transfer)
Initia->>Zaar: Establish IBC Channel (nft-transfer)
Zaar-->>Initia: Confirm Channel Connections
Possibly related PRs
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (2)
🚧 Files skipped from review as they are similar to previous changes (2)
⏰ Context from checks skipped due to timeout of 90000ms (1)
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Deploying registry-initiation-1 with Cloudflare Pages
|
Deploying registry-devnet with Cloudflare Pages
|
Deploying registry-testnet with Cloudflare Pages
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 3
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (2)
testnets/zaar/images/ZAAR.png
is excluded by!**/*.png
testnets/zaar/images/ZAAR.svg
is excluded by!**/*.svg
📒 Files selected for processing (5)
testnets/culinaris/chain.json
(1 hunks)testnets/initia/chain.json
(1 hunks)testnets/yominet/chain.json
(1 hunks)testnets/zaar/assetlist.json
(1 hunks)testnets/zaar/chain.json
(1 hunks)
✅ Files skipped from review due to trivial changes (1)
- testnets/yominet/chain.json
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: Cloudflare Pages: registry-initiation-1
🔇 Additional comments (4)
testnets/zaar/assetlist.json (1)
24-33
: Verify image assets existence.The image URLs reference PNG and SVG files. Ensure these files exist in the repository.
✅ Verification successful
✓ Image assets verified
Both PNG and SVG image files exist at the specified paths in the repository.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Check if the referenced image files exist for img in "testnets/zaar/images/ZAAR.png" "testnets/zaar/images/ZAAR.svg"; do if [ ! -f "$img" ]; then echo "Missing image file: $img" fi doneLength of output: 247
testnets/zaar/chain.json (1)
66-78
: Verify IBC channel configuration consistency.The IBC channels configuration matches with the channels added in initia/chain.json. This looks good.
testnets/culinaris/chain.json (1)
82-82
: LGTM! Executor URI follows the consistent pattern.The added executor_uri follows the same format as other testnets and matches the domain pattern used in other API endpoints.
testnets/initia/chain.json (1)
307-318
: LGTM! IBC channels are properly configured.The added IBC channels for Zaar testnet follow the consistent pattern:
- Correct port IDs (transfer, nft-transfer)
- Standard versions (ics20-1, ics721-1)
- Channel IDs follow the sequential numbering
Add Zaar
Add culinaris executor api
fix yominet asset list url
Summary by CodeRabbit
New Features
Documentation