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 Dev Tools multple data source integration feature #3653

Merged
merged 110 commits into from
Apr 25, 2023

Conversation

vagimeli
Copy link
Collaborator

@vagimeli vagimeli commented Apr 3, 2023

Description

Add new documentation on multidata source Dev Tools integration and create new TOC section Dev Tools

Issues Resolved

#3452

opensearch-project/OpenSearch-Dashboards#3577

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.

@vagimeli vagimeli requested a review from a team as a code owner April 3, 2023 20:12
@vagimeli vagimeli self-assigned this Apr 3, 2023
@vagimeli vagimeli added 2 - In progress Issue/PR: The issue or PR is in progress. Sev2 High-medium priority. Upcoming release or incorrect information. dashboards v2.7.0 labels Apr 3, 2023
@vagimeli vagimeli added this to the v2.7 milestone Apr 3, 2023
@vagimeli vagimeli linked an issue Apr 3, 2023 that may be closed by this pull request
4 tasks
@zhongnansu zhongnansu self-requested a review April 5, 2023 20:14
@vagimeli
Copy link
Collaborator Author

vagimeli commented Apr 5, 2023

@zhongnansu Please review this draft and make suggestions. Is any key information missing?

Copy link
Member

@zhongnansu zhongnansu left a comment

Choose a reason for hiding this comment

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

Overall lgtm. Left 1 non blocking comment. Thanks

_dashboards/discover/multi-data-sources.md Outdated Show resolved Hide resolved
@vagimeli
Copy link
Collaborator Author

vagimeli commented Apr 7, 2023

Overall lgtm. Left 1 non blocking comment. Thanks

@zhongnansu Revised to add steps 1 and 2 re: modifying YAML file

@vagimeli vagimeli added 4 - Doc review PR: Doc review in progress and removed 2 - In progress Issue/PR: The issue or PR is in progress. labels Apr 7, 2023
Copy link
Contributor

@cwillum cwillum left a comment

Choose a reason for hiding this comment

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

Looks good. Just suggestions.

_dashboards/dev-tools/index-dev.md Outdated Show resolved Hide resolved
_dashboards/discover/multi-data-sources.md Outdated Show resolved Hide resolved
_dashboards/discover/multi-data-sources.md Outdated Show resolved Hide resolved
_dashboards/discover/multi-data-sources.md Outdated Show resolved Hide resolved
_dashboards/discover/multi-data-sources.md Outdated Show resolved Hide resolved
_dashboards/discover/multi-data-sources.md Outdated Show resolved Hide resolved
_dashboards/discover/multi-data-sources.md Outdated Show resolved Hide resolved
Copy link
Contributor

@ariamarble ariamarble left a comment

Choose a reason for hiding this comment

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

One small change, otherwise looks good to me after Chris's comments

@vagimeli vagimeli added 5 - Editorial review PR: Editorial review in progress and removed 4 - Doc review PR: Doc review in progress labels Apr 12, 2023
@kolchfa-aws kolchfa-aws removed this from the v2.7 milestone Apr 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.

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

_dashboards/dev-tools/index-dev.md Outdated Show resolved Hide resolved
_dashboards/dev-tools/index-dev.md Outdated Show resolved Hide resolved
_dashboards/discover/multi-data-sources.md Outdated Show resolved Hide resolved
_dashboards/discover/multi-data-sources.md Outdated Show resolved Hide resolved
_dashboards/discover/multi-data-sources.md Outdated Show resolved Hide resolved
_dashboards/discover/multi-data-sources.md Outdated Show resolved Hide resolved
_dashboards/discover/multi-data-sources.md Outdated Show resolved Hide resolved
Signed-off-by: vagimeli <[email protected]>
@vagimeli vagimeli merged commit 74902a2 into main Apr 25, 2023
@vagimeli vagimeli deleted the issue-3452 branch April 25, 2023 21:06
@hdhalter hdhalter added the release-notes PR: Include this PR in the automated release notes label Apr 26, 2023
@vagimeli vagimeli added Closed - Complete Issue: Work is done and associated PRs closed and removed 6 - Done but waiting to merge PR: The work is done and ready to merge labels Apr 27, 2023
@vagimeli vagimeli changed the title [DOC] Add multidata source Dev Tools integration [DOC] Add documentation for Dev Tools multidata source integration feature Apr 27, 2023
Naarcha-AWS pushed a commit that referenced this pull request Apr 27, 2023
* Create new doc site section Dev Tools

Signed-off-by: vagimeli <[email protected]>
@hdhalter hdhalter changed the title [DOC] Add documentation for Dev Tools multidata source integration feature Add documentation for Dev Tools multple idata source integration feature May 2, 2023
@hdhalter hdhalter changed the title Add documentation for Dev Tools multple idata source integration feature Add documentation for Dev Tools multple data source integration feature May 2, 2023
vagimeli added a commit that referenced this pull request May 4, 2023
* Create new doc site section Dev Tools

Signed-off-by: vagimeli <[email protected]>
vagimeli added a commit that referenced this pull request May 4, 2023
harshavamsi pushed a commit to harshavamsi/documentation-website that referenced this pull request Oct 31, 2023
…3653)

* Create new doc site section Dev Tools

Signed-off-by: vagimeli <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Closed - Complete Issue: Work is done and associated PRs closed dashboards release-notes PR: Include this PR in the automated release notes Sev2 High-medium priority. Upcoming release or incorrect information. v2.7.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[DOC] OSD Multiple Datasource integration features for GA release