-
Notifications
You must be signed in to change notification settings - Fork 83
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
Audit config section, populate advanced-config section #167
Conversation
✅ Deploy Preview for lotus-docs ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
@TippyFlitsUK @rjan90 @Reiers Can you please add information to the logging page? I need help with Rust logs. It is marked as TODO. |
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.
Very nice work Mayank!! Just waiting on confirmation of the RUST logging steps (testing now).
Co-authored-by: TippyFlits <[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.
Changes needed:
- When mentioning the Lotus miner application, always surround it in quotes and add a dash. So
lotus-miner
, not Lotus miner. - Don't use quote blocks for call outs.
- If files have moved around you need to add redirects/aliases.
There are a lot of changes in this PR. Next time, split your changes between multiple smaller PRs, it makes things much easier to review and figure out what's going wrong when something breaks.
content/en/storage-providers/advanced-configurations/logging.md
Outdated
Show resolved
Hide resolved
content/en/storage-providers/advanced-configurations/logging.md
Outdated
Show resolved
Hide resolved
content/en/storage-providers/advanced-configurations/logging.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Johnny <[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.
Thanks for those changes!
Fixes parts of #77
Empty config section
Check links
Check aliases are in place
Rename config section to Setup
Add pre-requisite page
Make changes in menu
Do final check for 404 or incorrect redirects
Populate logging page - Separate rust log file