Skip to content
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

Adds documentation for Admin UI index operations #2403

Merged
merged 15 commits into from
Jan 17, 2023
Merged

Adds documentation for Admin UI index operations #2403

merged 15 commits into from
Jan 17, 2023

Conversation

ariamarble
Copy link
Contributor

Signed-off-by: ariamarble [email protected]

Description

Adds documentation for Admin UI index operations

Issues Resolved

#2306

Checklist

  • By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
    For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@ariamarble ariamarble requested a review from a team as a code owner January 13, 2023 00:07
@ariamarble ariamarble self-assigned this Jan 13, 2023
@ariamarble ariamarble added index-management 3 - Tech review PR: Tech review in progress v2.5.0 'Issues and PRs related to version v2.5.0' labels Jan 13, 2023
@ariamarble
Copy link
Contributor Author

Sending for tech review, will be working on adding links to SSs

@Hailong-am
Copy link
Contributor

images/admin-ui-index/shrink.png may need to refreshed, we have remove the warning message.

@SuZhou-Joe
Copy link
Member

SuZhou-Joe commented Jan 13, 2023

Some screenshots contain some test data, Would you like to change for screenshots with pure initial data?

  • aliases.png
    image
  • templates.png
    image

@SuZhou-Joe
Copy link
Member

For the create-index-template-expanded.png, I'd suggest that expand the advanced settings as well.
image

@ariamarble
Copy link
Contributor Author

@SuZhou-Joe I have lost access to the test env. (bad gateway) so I'll grab new screenshots when I can.

@ariamarble ariamarble added 4 - Doc review PR: Doc review in progress and removed 3 - Tech review PR: Tech review in progress labels Jan 13, 2023
@hdhalter hdhalter added the release-notes PR: Include this PR in the automated release notes label Jan 13, 2023
_dashboards/admin-ui-index/index-management.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/index-management.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/index-management.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/index-management.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/index-management.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/index-management.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/index-management.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/index-management.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/index-management.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/index-management.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/index-management.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/index-management.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/index-management.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/index-management.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/index-management.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/index-management.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/index-management.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/index-management.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/index-management.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/snapshot-management.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/index-management.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/index-management.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/index-management.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/index-management.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/index-management.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/index-management.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/snapshot-management.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@kolchfa-aws kolchfa-aws left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with comments

_dashboards/admin-ui-index/index-management.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/sm-dashboards.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/sm-dashboards.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/sm-dashboards.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/sm-dashboards.md Outdated Show resolved Hide resolved
Copy link
Contributor

@vagimeli vagimeli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minimal comments; LGTM

Good work on this doc!

_dashboards/admin-ui-index/sm-dashboards.md Outdated Show resolved Hide resolved
_dashboards/admin-ui-index/sm-dashboards.md Outdated Show resolved Hide resolved
@ariamarble ariamarble removed the 4 - Doc review PR: Doc review in progress label Jan 17, 2023
@ariamarble ariamarble merged commit 2feb50e into main Jan 17, 2023
Copy link
Collaborator

@natebower natebower left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ariamarble Please see my comments and let me know if you have any questions. Thanks!

_dashboards/admin-ui-index/sm-dashboards.md Show resolved Hide resolved
_dashboards/admin-ui-index/sm-dashboards.md Show resolved Hide resolved
Copy link
Collaborator

@natebower natebower left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ariamarble Please see my comments and let me know if you have any questions. Thanks!

vagimeli pushed a commit that referenced this pull request Jan 25, 2023
* Adds documentation for Admin UI index operations

Signed-off-by: ariamarble <[email protected]>

* tech review and format changes

Signed-off-by: ariamarble <[email protected]>

* Update _dashboards/admin-ui-index/index-management.md

Co-authored-by: kolchfa-aws <[email protected]>

* Update _dashboards/admin-ui-index/index-management.md

Co-authored-by: kolchfa-aws <[email protected]>

* Update _dashboards/admin-ui-index/index-management.md

Co-authored-by: kolchfa-aws <[email protected]>

* Update _dashboards/admin-ui-index/index-management.md

Co-authored-by: kolchfa-aws <[email protected]>

* Update _dashboards/admin-ui-index/index-management.md

Co-authored-by: kolchfa-aws <[email protected]>

* Update _dashboards/admin-ui-index/index-management.md

Co-authored-by: kolchfa-aws <[email protected]>

* Update _dashboards/admin-ui-index/index-management.md

Co-authored-by: kolchfa-aws <[email protected]>

* Update _dashboards/admin-ui-index/index-management.md

Co-authored-by: kolchfa-aws <[email protected]>

* Apply suggestions from code review

Co-authored-by: kolchfa-aws <[email protected]>

* initial doc review changes

Signed-off-by: ariamarble <[email protected]>

* further doc review changes

Signed-off-by: ariamarble <[email protected]>

* consolidate snapshots

Signed-off-by: ariamarble <[email protected]>

* Doc Review 2 Electric Boogaloo

Signed-off-by: ariamarble <[email protected]>

Signed-off-by: ariamarble <[email protected]>
Co-authored-by: kolchfa-aws <[email protected]>
vagimeli added a commit that referenced this pull request Jan 26, 2023
@hdhalter hdhalter deleted the issue2306 branch March 28, 2024 21:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
index-management release-notes PR: Include this PR in the automated release notes v2.5.0 'Issues and PRs related to version v2.5.0'
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants