docs: subnet types reference#65
Conversation
98b0952 to
d25cd7d
Compare
Review: PR #65 — reference/subnet-types.mdMust fix
Suggestions
Verified
|
Review: Subnet Types ReferenceMust fix
Suggestions
Verified
|
- Fix ambiguous storage phrasing: "subnets share a 2 TiB storage budget" → "each application subnet has a 2 TiB storage capacity" - Update uzr34 subnet label from "Application services" to "Internet Identity / infrastructure" to match portal source - Replace hardcoded fiduciary node count (34) with ICP Dashboard reference — sources conflict (portal/references says 31, portal/guides says 34) so direct readers to the authoritative live source - Add verification note for fiduciary principal ID in deploy command (two portal sources use different suffixes; icp-cli version kept as most authoritative for CLI docs, but note added to verify on dashboard) - Update cycle cost multiplier formula to be generic (n/13) rather than hardcoding 34 nodes - Improve summary table system subnet row with known uzr34 node count (28) - Update fiduciary canister creation cost row to reference dashboard instead of hardcoded cycle count - Add USD exchange rate caveat to cost section
|
<!-- feedback-addressed --> Feedback addressed — PR #65 (docs/reference/subnet-types.md)Changes appliedMust-fix items:
Suggestion items applied:
Items skipped
|
Summary
tdb26), application services (uzr34), Bitcoin (w4rem) — no cycles charged, user canisters not allowedpzp6e— hosts threshold signing keys and EVM RPC canisterbkfrj— for GDPR-alignmentSync recommendation
informed by
dfinity/portal—docs/building-apps/developing-canisters/deploy-specific-subnet.mdx; informed bydfinity/icp-cli—docs/guides/deploying-to-specific-subnets.md