Migrate reporting frontend plugin to OpenSearch Dashboards #4
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
dblock
requested changes
Apr 16, 2021
dashboards-reports/README.md
Outdated
| @@ -0,0 +1,70 @@ | |||
| # OpenSearch Dashboards Reports for Open Distro | |||
There was a problem hiding this comment.
Are we leaving "Open Distro" here on purpose?
There was a problem hiding this comment.
missed this.. updated
dashboards-reports/README.md
Outdated
|
|
||
| ## Setup | ||
|
|
||
| 1. Download Elasticsearch for the version that matches the [OpenSearch Dashboards version specified in package.json](./package.json#L7). |
dashboards-reports/README.md
Outdated
|
|
||
| ## Copyright | ||
|
|
||
| Copyright 2020 Amazon.com, Inc. or its affiliates. All Rights Reserved. |
There was a problem hiding this comment.
2021, like in opensearch-project/OpenSearch@807d0e3
dashboards-reports/THIRD-PARTY
Outdated
| * For dashboards-reports see also this required NOTICE: | ||
| dashboards-reports | ||
| Copyright 2020 Amazon.com, Inc. or its affiliates. All Rights Reserved. |
dashboards-reports/server/types.ts
Outdated
| @@ -14,6 +14,6 @@ | |||
| */ | |||
|
|
|||
| // eslint-disable-next-line @typescript-eslint/no-empty-interface | |||
| export interface OpendistroKibanaReportsPluginSetup {} | |||
| export interface OpendistroReportsOpenSearchDashboardsPluginSetup {} | |||
There was a problem hiding this comment.
More opendistro here and the likes.
There was a problem hiding this comment.
Changed to ReportsDashboards, thanks
dblock
approved these changes
Apr 16, 2021
19 tasks
kavilla
pushed a commit
to kavilla/dashboards-reports
that referenced
this pull request
Jul 12, 2023
…search-project#4) * Migrate to Opensearch * Rename Kibana in html and md files * WIP rename strings and variables * Rename kibana index * revert kibana_recipients temporarily * Change kbn to osd * Update jest snapshots * Migrate cypress * Change opendistro_kibana_reports to opendistro-reports-opensearch-dashboards * fixup! Migrate to Opensearch * Typo fixes and more renaming * Rename variables in delivery params * Rename kibana user to opensearch dashboards user * Rename Es in variables * Migrate github workflow for dashboards-reports * Move kibana-reports dir to dashboards-reports * Migrate readme * Rename es to opensearch * Remove opendistro in dashboards reports * Update documents for repo * Address comments
kavilla
pushed a commit
to kavilla/dashboards-reports
that referenced
this pull request
Jul 12, 2023
…roject#6) * Add necessary files (opensearch-project#4) * Add docs Signed-off-by: Rupal Mahajan <[email protected]> * Update docs to repo name change Signed-off-by: Rupal Mahajan <[email protected]> * Updated MAINTAINERS.md to match recommended opensearch-project format. Signed-off-by: Rupal Mahajan <[email protected]> Signed-off-by: Rupal Mahajan <[email protected]> * Remove chromium related info from readme Signed-off-by: Rupal Mahajan <[email protected]> * nit: remove troubleshooting section Signed-off-by: Rupal Mahajan <[email protected]> * nit: missed font dependencies under troubleshooting Signed-off-by: Rupal Mahajan <[email protected]> Signed-off-by: Rupal Mahajan <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue #, if available:
Description of changes:
TODO left:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.