-
Notifications
You must be signed in to change notification settings - Fork 916
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
[Rename] visualizations plugin #159
[Rename] visualizations plugin #159
Conversation
@@ -393,7 +393,7 @@ export const buildPipeline = async (vis: Vis, params: BuildPipelineParams) => { | |||
const { uiState, title } = vis; | |||
|
|||
// context | |||
let pipeline = `kibana | kibana_context `; | |||
let pipeline = `opensearch_dashboards | opensearch_dashboards_context `; |
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.
Is this in expression context ? if so @boktorbb-amzn has replaced with opensearchDashboards
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.
looks like it. we should sync up about that or w/e gets merged in.
src/plugins/visualizations/server/saved_objects/visualization_migrations.ts
Outdated
Show resolved
Hide resolved
src/plugins/visualizations/server/saved_objects/visualization_migrations.ts
Outdated
Show resolved
Hide resolved
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.
Few changes , rest looks good.
src/plugins/visualizations/server/saved_objects/visualization_migrations.ts
Outdated
Show resolved
Hide resolved
src/plugins/visualizations/server/saved_objects/visualization_migrations.ts
Outdated
Show resolved
Hide resolved
Signed-off-by: Kawika Avilla <[email protected]>
Signed-off-by: Kawika Avilla <[email protected]>
Signed-off-by: Kawika Avilla <[email protected]>
91867d9
to
896fda3
Compare
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.
LGTM
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.
LGTM !!
* [Rename] visualizations plugin Signed-off-by: Kawika Avilla <[email protected]> * Forgot to save rename Signed-off-by: Kawika Avilla <[email protected]> * Address PR comments Signed-off-by: Kawika Avilla <[email protected]>
* [Rename] visualizations plugin Signed-off-by: Kawika Avilla <[email protected]> * Forgot to save rename Signed-off-by: Kawika Avilla <[email protected]> * Address PR comments Signed-off-by: Kawika Avilla <[email protected]>
* [Rename] visualizations plugin Signed-off-by: Kawika Avilla <[email protected]> * Forgot to save rename Signed-off-by: Kawika Avilla <[email protected]> * Address PR comments Signed-off-by: Kawika Avilla <[email protected]>
Signed-off-by: Yulong Ruan <[email protected]>
…) (opensearch-project#169) (cherry picked from commit 1d95a09) Signed-off-by: Yulong Ruan <[email protected]> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
…roject#159)" This reverts commit 47e10e4.
…roject#159)" This reverts commit 47e10e4.
Issue #, if available:
#37
Description of changes:
Rename files and variables within plugins/visualizations
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.