-
Notifications
You must be signed in to change notification settings - Fork 272
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
Global review #259
Global review #259
Conversation
✔️ Deploy Preview for substrate-docs ready! 🔨 Explore the source changes: 9115582 🔍 Inspect the deploy log: https://app.netlify.com/sites/substrate-docs/deploys/6163be8699c48c0007b4538c 😎 Browse the preview: https://deploy-preview-259--substrate-docs.netlify.app |
✔️ Deploy Preview for substrate-docs-staging ready! 🔨 Explore the source changes: 9115582 🔍 Inspect the deploy log: https://app.netlify.com/sites/substrate-docs-staging/deploys/6163be86808c7a0008afced7 😎 Browse the preview: https://deploy-preview-259--substrate-docs-staging.netlify.app |
# Conflicts: # v3/how-to-guides/07-parachains/c-reserve-id/index.mdx # v3/how-to-guides/07-parachains/f-start-testnet/index.mdx
Just wanted to check in on this - were the TODOs fixed in this PR, or just removed? If the latter, let's please highlight them in #179 |
@nukemandan : The TODOS for parachain how-to guides that were removed were in relation to adding Examples and References. There are still some that were left in (global search: |
I don't think having 2 issues for the same thing is necessary. In this case, we already have issues open about completing parachain content. It turns out the only FIXMEs / TODOs we have outstanding are for that content AFAIK. |
This PR is a general pass on content to address miscellaneous issues, including:
SignedExtension
part of the extrinsics doc? #156 : I moved the Signed Extension section out of "Extrinsics" and into "Transaction Pool"