-
Notifications
You must be signed in to change notification settings - Fork 321
chore: update Admin UI doc to reflect the new UI design #6528
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
Conversation
157d579 to
84b196b
Compare
68b17b4 to
095592a
Compare
| - [Access the Admin UI](#access-the-admin-ui) | ||
| - [View account information](#view-account-information) | ||
| - [View cluster information](#view-cluster-information) | ||
| - [Access operational dashboards](#access-operational-dashboards) |
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.
The 'operational dashboards' and 'management tokens' anchors are the ones that weren't on this page (so existing links didn't work)
|
|
||
| After you log in, the Account Management portal displays [account information](/influxdb3/cloud-dedicated/admin/account/) | ||
| and lists all clusters associated with your account. | ||
| 3. **Search** for the cluster or use the sort button and column headers to sort the cluster list and find the cluster. |
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.
We could leave all of this in, but I found this a little repetitive; for purposes of opening the application I think providing the URL and request to contact support if credentials are required is sufficient.
| > #### Database names can't be updated | ||
| > | ||
| > [!Note] | ||
| > #### Database names can't be updated with this command |
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 edited this because there is a database rename command in influxctl now.
| - **Time Range Configuration**: Adjust the metrics time range | ||
| - **Live Updates**: Enable automatic refresh for real-time monitoring | ||
|
|
||
| ## Monitor with Grafana operational dashboards |
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.
@ritwika314 I imagine we should keep the Grafana operational dashboards instructions at least for now.
|
|
||
| 1. Access the {{% product-name %}} Admin UI at [console.influxdata.com](https://console.influxdata.com). | ||
| If you don't have login credentials, [contact InfluxData support](https://support.influxdata.com). | ||
| 2. Log in using the credentials provided by InfluxData. |
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 proposed condensing this throughout as I thought it might be too wordy; I'll revert if we think the specific instructions on selecting the account/cluster are needed on each page
| @@ -1,100 +0,0 @@ | |||
| --- | |||
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 doesn't delete this material, but rather moves it over into the body of the docs.
095592a to
365a1c1
Compare
|
fyi @blegesse-w @ritwika314: I added a separate commit implementing what I understand has been discussed with @jstirnaman about the organization of the Admin UI documentation going forward: i.e., removing the old 'reference' section, and moving the Admin UI documentation and screenshots into the main documentation sections (which allow choosing between the Admin UI, influxctl, and sometimes the management API, depending on the context). |
|
One thing I was not sure about how to resolve: I did get several "Vale" errors of the following form in my local instance of the repo, which I think are due to the inclusion of URLs and code blocks including the term 'API' (and commas and periods). I ignored the warnings on commit/push, but if I should resolve locally, please advise. |
sanderson
left a comment
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.
Some minor punctuation nits, but otherwise looks great! Thanks @blegesse-w and @wdoconnell
ritwika314
left a comment
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 mostly just propagated @sanderson's punctuation suggestions to other lists, looks great overall, thank you @blegesse-w and @wdoconnell !
| Access the InfluxDB Cloud Dedicated Admin UI at [console.influxdata.com](https://console.influxdata.com). | ||
| If you don't have login credentials, [contact InfluxData support](https://support.influxdata.com). |
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.
@sanderson could this be a macro/shortcut? its repeated on every ui page.
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.
@ritwika314 It is now! I added it with some templating.
content/influxdb3/cloud-dedicated/admin/tokens/database/list.md
Outdated
Show resolved
Hide resolved
content/influxdb3/cloud-dedicated/admin/tokens/management/list.md
Outdated
Show resolved
Hide resolved
content/influxdb3/cloud-dedicated/admin/monitor-your-cluster.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Scott Anderson <sanderson@users.noreply.github.com>
Co-authored-by: Scott Anderson <sanderson@users.noreply.github.com>
Co-authored-by: Scott Anderson <sanderson@users.noreply.github.com>
Co-authored-by: Scott Anderson <sanderson@users.noreply.github.com>
Co-authored-by: ritwika314 <mim.ghosh@gmail.com>
Co-authored-by: ritwika314 <mim.ghosh@gmail.com>
Co-authored-by: ritwika314 <mim.ghosh@gmail.com>
Co-authored-by: ritwika314 <mim.ghosh@gmail.com>
Co-authored-by: ritwika314 <mim.ghosh@gmail.com>
False positives - fixing those rules is on my todo list. |
PR updates admin UI doc to reflect the new UI design
Major Updates:
New Navigation Section - added "Navigate the Admin UI" to explain the collapsible sidebar structure
Account and Cluster Management - expanded section explaining the new hierarchical navigation with account/cluster selectors at the bottom of the sidebar
All Accounts/Clusters Views - added documentation for viewing all accounts and clusters
Cluster Overview
All pages now reference the sidebar navigation instead of the old portal-style navigation
Added new section documenting the Help menu with links to documentation, support, and sales
Management Tokens
Signed the InfluxData CLA
(if necessary)
Rebased/mergeable