-
Notifications
You must be signed in to change notification settings - Fork 142
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
[DOCS] General improvements - part1 #3645
Conversation
…to Formatting_issues
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 Job, just a few small comments
Steps in the UI: | ||
{.power-number} | ||
|
||
1. Go to the main menu and select **Operating System (OS) > Overview**. | ||
|
||
2. In the **Node Names** drop-down menu, choose the new node you want to monitor. | ||
|
||
3. Modify the time range to view the relevant data for your selected 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.
this part seems like it's possible to add node in UI.
especially choose the new node you want to monitor
maybe we should rewrite it like select new node you just registered/installed
?
And maybe replace Steps in the UI
to How to see monitored node in UI
, Find your node in UI
or something like that.
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.
updated section
@@ -489,6 +489,6 @@ For more information, see [PMM Dashboards](../use/dashboards-panels/index.md). | |||
## Next steps | |||
|
|||
- [Configure PMM via the interface](../configure-pmm/configure.md) | |||
- [Manage users in PMM](../how-to/manage-users.md) | |||
- [Manage users in PMM](../pmm-admin/manage-users/index.md) |
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.
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.
ok, I've renamed the folder. Thanks!
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.
One more question: are we sure to drop the Trademark Policy?
|
||
 | ||
 |
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 would recommend we set the alt
tag to something meaningful rather than leave it empty to honor SEO :)
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.
fixed in this topic, will update all images in another PR with such improvements
It was a duplicate topic |
PMM-0
Link to the Feature Build: SUBMODULES-0
If this PR adds or removes or alters one or more API endpoints, please review and add or update the relevant API documents as well:
If this PR is related to some other PRs in this or other repositories, please provide links to those PRs: