-
Notifications
You must be signed in to change notification settings - Fork 11
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
[Polkadot Wiki Migration] Set up a Bootnode #34
Conversation
CrackTheCode016
commented
Sep 17, 2024
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.
This section's current structure lacks logical flow for developers. We could reorganize it and add more step-by-step guidance.
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.
I made most of the grammar type edits but there are a couple here as requested changes. @nhussein11 or @0xLucca please see comments about content needs. Thanks!
|
||
Before you start, you need to have the following prerequisites: | ||
|
||
<!-- TODO: What goes here? --> |
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.
@0xLucca or @nhussein11: please add the appropriate items here. We have a nice example in tutorials/polkadot-sdk/build-a-blockchain/add-trusted-nodes.md
on #29 so you can see what I mean. Thank you!
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.
I added a few, but please see if it needs to change.
Co-authored-by: Dawn Kelly <[email protected]>
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.
lgtm
…dot-docs into bd-infra-setup-boot-node
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.
LGTM from formatting perspective
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.
LGTM!