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

Add documentation for dashboards assistant plugin #6370

Merged

Conversation

ruanyl
Copy link
Member

@ruanyl ruanyl commented Feb 8, 2024

Description

Documentation for dashboards assistant plugin.

The documentation is created by @xluo-aws @wanglam @gaobinlong @raintygao @ruanyl

Issues Resolved

Closes #6369

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.

Signed-off-by: Yulong Ruan <ruanyl@amazon.com>
@kgcreative
Copy link
Member

This should be "OpenSearch Assistant Toolkit for OpenSearch Dashboards" (if we are specifically referring to the OpenSearch Dashboards features, as opposed to "OpenSearch Assistant Toolkit" for all the backend tooling)

@hdhalter hdhalter added release-notes PR: Include this PR in the automated release notes v2.12.0 4 - Doc Review PR: Doc review in progress labels Feb 8, 2024
Copy link
Collaborator

@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.

@ruanyl Please review my edits and comments. Please make sure my edits have retained technical accuracy. Once you approves the changes, I'll format the file and replace any low-resolution screenshots. Please provide the testing environment. Following this, I'll schedule the doc for editorial review. Please be available to answer any technical questions. Thank you, Melissa

cc: @nitincd @kgcreative for input
cc: @kolchfa-aws for doc website structure

_dashboards-assistant/index.md Outdated Show resolved Hide resolved
_dashboards-assistant/index.md Outdated Show resolved Hide resolved
_dashboards-assistant/index.md Outdated Show resolved Hide resolved
_dashboards-assistant/index.md Outdated Show resolved Hide resolved
_dashboards-assistant/index.md Outdated Show resolved Hide resolved
_dashboards-assistant/index.md Outdated Show resolved Hide resolved
_dashboards-assistant/index.md Outdated Show resolved Hide resolved
_dashboards-assistant/index.md Outdated Show resolved Hide resolved
_dashboards-assistant/index.md Outdated Show resolved Hide resolved
_dashboards-assistant/index.md Outdated Show resolved Hide resolved
@vagimeli
Copy link
Collaborator

vagimeli commented Feb 9, 2024

This should be "OpenSearch Assistant Toolkit for OpenSearch Dashboards" (if we are specifically referring to the OpenSearch Dashboards features, as opposed to "OpenSearch Assistant Toolkit" for all the backend tooling)

@kgcreative I agree. I've changed doc title.

@vagimeli
Copy link
Collaborator

@xluo-aws @wanglam @gaobinlong @raintygao @ruanyl

To move this documentation forward, please provide your technical review and approval. Once I have your approval of my rewrites and edits, I can move the content into editorial review and then merge the PR. Please reach out to me with any questions. Thank you.

@vagimeli vagimeli added 3 - Tech Review PR: Tech review in progress and removed 4 - Doc Review PR: Doc review in progress labels Feb 13, 2024
@ylwu-amzn
Copy link
Contributor

@ruanyl looks some screenshot should be updated

<img width="600px" src="{{site.url}}{{site.baseurl}}/images/dashboards-assistant/suggestions.png" alt="The assistant suggestions">

### Rate the response
Each response contains a thumb up and thumb down icon button, you can click on the button to feedback whether you are satisfied with the response.
Copy link
Member

Choose a reason for hiding this comment

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

We should add a little tidbit on where this feedback is stored, so users won't have concerns that the data is not being transmitted anywhere?

@xluo-aws
Copy link
Member

Will update the doc by next Monday if that's ok. All related team members are taking holidays off this week.

@raintygao
Copy link

raintygao commented Feb 15, 2024

Will update the doc by next Monday if that's ok. All related team members are taking holidays off this week.

@xluo-aws According to release issue, pre release is on the end of February 19th. If there are any updates needed we can complete them before end of 18th, not sure if it's OK.

@raintygao
Copy link

@ruanyl looks some screenshot should be updated

@ylwu-amzn Could you elaborate more?

ruanyl and others added 2 commits February 15, 2024 19:29
Co-authored-by: Melissa Vagi <vagimeli@amazon.com>
Signed-off-by: Yulong Ruan <ruanyu1@gmail.com>
Co-authored-by: Melissa Vagi <vagimeli@amazon.com>
Signed-off-by: Yulong Ruan <ruanyu1@gmail.com>
vagimeli and others added 5 commits February 15, 2024 16:19
Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
Signed-off-by: Yulong Ruan <ruanyl@amazon.com>
Signed-off-by: Yulong Ruan <ruanyl@amazon.com>
@natebower natebower added 6 - Done but waiting to merge PR: The work is done and ready to merge and removed 5 - Final Editorial Review PR: Editorial Review in progress labels Feb 16, 2024
@ylwu-amzn
Copy link
Contributor

@ruanyl looks some screenshot should be updated

@ylwu-amzn Could you elaborate more?

Shared you a latest test cluster. You can take new screenshots there.

Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
@vagimeli
Copy link
Collaborator

@ruanyl looks some screenshot should be updated

@ylwu-amzn Could you elaborate more?

Shared you a latest test cluster. You can take new screenshots there.

@ylwu-amzn @ruanyl I've replaced screenshots based on the the test environment you shared. Please provide your approval of this PR if its ready for merging. Thank you, Melissa

```
{% include copy-curl.html %}

This example is a system index. In security-enabled domains, only super administrators have access to execute this code. For details about making super admin calls, see the [System indexes]({{site.url}}{{site.baseurl}}/security/configuration/system-indices/) guide. For access permission, contact your IT administrator.
Copy link
Collaborator

Choose a reason for hiding this comment

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

@natebower Please edit lines 43-44.

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.

@vagimeli Line 43 edited.

_dashboards/dashboards-assistant/index.md Outdated Show resolved Hide resolved
```
{% include copy-curl.html %}

This example is a system index. In security-enabled domains, only super administrators have access to execute this code. For details about making super admin calls, see the [System indexes]({{site.url}}{{site.baseurl}}/security/configuration/system-indices/) guide. For access permission, contact your IT administrator.
Copy link
Collaborator

Choose a reason for hiding this comment

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

This example shows/provides/uses a system index? (The example isn't literally a system index). Also, I believe I may have seen "superadmin" used elsewhere. I would confirm what we've used elsewhere and make consistent here.

vagimeli and others added 6 commits February 19, 2024 11:32
Co-authored-by: Nathan Bower <nbower@amazon.com>
Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
Signed-off-by: Melissa Vagi <vagimeli@amazon.com>

Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
Signed-off-by: Melissa Vagi <vagimeli@amazon.com>

Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
Signed-off-by: Melissa Vagi <vagimeli@amazon.com>

Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
Copy link
Collaborator

@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.

Doc and editorial reviews completed. Editorial feedback addressed.

Signed-off-by: Melissa Vagi <vagimeli@amazon.com>

Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
@vagimeli vagimeli added 3 - Done Issue is done/complete and removed 6 - Done but waiting to merge PR: The work is done and ready to merge labels Feb 19, 2024
@vagimeli vagimeli merged commit f730f7d into opensearch-project:main Feb 19, 2024
3 of 4 checks passed
oeyh pushed a commit to oeyh/documentation-website that referenced this pull request Mar 14, 2024
…#6370)

* Add documentation for dashboards assistant plugin

---------

Signed-off-by: Yulong Ruan <ruanyl@amazon.com>
Signed-off-by: Yulong Ruan <ruanyu1@gmail.com>
Signed-off-by: Melissa Vagi <vagimeli@amazon.com>
Co-authored-by: Melissa Vagi <vagimeli@amazon.com>
Co-authored-by: Nathan Bower <nbower@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 - Done Issue is done/complete experimental release-notes PR: Include this PR in the automated release notes v2.12.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[DOC] Instruction to use chatbot (Dashboards Assistant)
10 participants