-
Notifications
You must be signed in to change notification settings - Fork 78
Migrate XCM replay page #1341
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
base: eshaben/enable-ci
Are you sure you want to change the base?
Migrate XCM replay page #1341
Conversation
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.
Pull request overview
This PR migrates and improves the XCM replay documentation by enhancing clarity, updating references, and standardizing formatting. The changes focus on making the guide more accessible and consistent while maintaining technical accuracy.
Key changes:
- Updated Chopsticks documentation references and streamlined the introduction section
- Standardized terminology (e.g., "behavior" instead of "behaviour") and improved bullet point formatting
- Enhanced code block annotations with proper titles and clarified setup instructions
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
chain-interactions/send-transactions/interoperability/debug-and-preview-xcms.md
Show resolved
Hide resolved
chain-interactions/send-transactions/interoperability/debug-and-preview-xcms.md
Outdated
Show resolved
Hide resolved
chain-interactions/send-transactions/interoperability/debug-and-preview-xcms.md
Outdated
Show resolved
Hide resolved
chain-interactions/send-transactions/interoperability/debug-and-preview-xcms.md
Outdated
Show resolved
Hide resolved
brunopgalvao
left a comment
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.
For the "Initialize a new Node project:" section, I think you need to add && npm pkg set type=module for the package.json to use module.
brunopgalvao
left a comment
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.
Let's refactor this guide to reference Polkadot Hub as much as possible instead of Asset Hub.
|
@nhussein11 wasn't fully sure what I could change, like if I could change code elements that say asset hub to polkadot hub, so I didn't make any changes on that front. But, the rest of the guide is in good shape. |
… documentation for consistency.
…g-and-preview-xcms.md
@brunopgalvao @ericmorel thanks! fixed in 10399e7 |
📝 Description
This pull request updates and improves the documentation for replaying and dry-running XCMs using Chopsticks. The changes focus on clarifying instructions, improving consistency, updating references, and enhancing the overall readability of the guide.
Documentation improvements and clarifications:
Consistency and language improvements:
🔍 Review Preference
Choose one:
🤖 AI-Ready Docs
If content changed, regenerate AI files:
python3 scripts/generate_llms.py✅ Checklist