-
-
Notifications
You must be signed in to change notification settings - Fork 120
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
[monitoring-docs] Restructured Documentation: New Unified Documentation Website #585
Conversation
docstrfmt -l 74
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.
Great work @pandafy, see my comments below.
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.
Building now from scratch raises the following warnings for me:
/developer/utils.rst:415: WARNING: undefined label: 'checks'
/user/checks.rst:57: WARNING: undefined label: 'adding checks and alert settings from the device page'
/user/intro.rst:6: WARNING: unknown document: 'dashboard_monitoring_charts'
Please double check.
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.
[skip ci]
[skip ci]
[skip ci]
[skip ci]
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.
Ready
Related to openwisp/openwisp-docs#107.
These documents are pulled by the logic being worked on in openwisp/openwisp-docs#189.
We are doing similar changes in all modules.