Announce Nano (XNO) availability across documentation and templates#218
Open
cbrunnkvist wants to merge 1 commit into
Open
Announce Nano (XNO) availability across documentation and templates#218cbrunnkvist wants to merge 1 commit into
cbrunnkvist wants to merge 1 commit into
Conversation
Add Nano to every chain enumeration across docs, README, templates, partials, and the ows skill — including the canonical supported-chains spec, HD derivation tree, shorthand alias table, and all generated binding READMEs (regenerated via readme/generate.sh).
|
@cbrunnkvist is attempting to deploy a commit to the MoonPay Team on Vercel. A member of the Team first needs to authorize it. |
Contributor
Author
|
Uh, also please let me know if completely overlooked how perhaps some docs-amending agent maybe is supposed do do this authoring in some semi-auto fashion?! I.e. am I even doing the right thing here? |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I was a bit too focused on the technical side in #109, and left out a fair chunk of "end-user discoverability" by failing to mention the chain in all (more than I expected) the different places where we mention builtin support for x y and z.
This PR ensures that Nano/XNO/
nano_are consistently listed alongside other supported chains, what standard derivation paths are used, etc. There are also some code samples updated accordingly...Mostly, I found mentions in...:
README.mdfilesIn short, we ensure that developers, users and Googlebots alike can discover that Nano is now a supported chain in the OWS ecosystem 👍
Testing
cargo test --workspacepassescargo clippy --workspace -- -D warningsis cleannpm testpasses (if Node bindings changed)owsCLINotes
There is already a companion-utility that bundles a set of Nano-transacting Agent skills 🦀 at @casualsecurityinc/xno-skills now built directly on top of the OWS 🥳 (but I think it is outside of the scope of these OWS documents to mention 3rd party tooling in the docs)
Note
Low Risk
Documentation-only updates that list Nano as a supported chain; no runtime logic changes, so risk is limited to potential docs inconsistencies.
Overview
Adds Nano (XNO) to the documented supported-chain set across the main README, Node/Python binding READMEs, crate READMEs, and generated readme templates/partials.
Updates chain tables and examples to include Nano-specific details (coin type
165, derivation pathm/44'/165'/0',nano_address prefix), and extendsdocs/07-supported-chains.mdwithnano:mainnetand alias/HD-derivation references; quickstart text is also updated to mention Nano in the default chain set.Reviewed by Cursor Bugbot for commit 49d2cbc. Bugbot is set up for automated code reviews on this repo. Configure here.