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

change samples to saved object #427

Merged
merged 9 commits into from
Apr 27, 2023

Conversation

derek-ho
Copy link
Collaborator

@derek-ho derek-ho commented Apr 25, 2023

Description

Saves sample panels as saved object instead of observability ops panel

Issues Resolved

[List any issues this PR will resolve]

Check List

  • New functionality includes testing.
    • All tests pass, including unit test, integration test and doctest
  • New functionality has been documented.
    • New functionality has javadoc added
    • New functionality has user manual doc added
  • Commits are signed per the DCO using --signoff

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

Signed-off-by: Derek Ho <[email protected]>
Signed-off-by: Derek Ho <[email protected]>
Signed-off-by: Derek Ho <[email protected]>
Signed-off-by: Derek Ho <[email protected]>
Signed-off-by: Derek Ho <[email protected]>
@@ -50,6 +50,8 @@ export const isNameValid = (name: string) => {
return name.length >= 50 || name.length === 0 ? false : true;
};

export const samplePanelName = '[Logs] Web traffic Panel';
Copy link
Member

Choose a reason for hiding this comment

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

minor: should we move this to common/constants?

import {
addMultipleVisualizations,
addVisualizationPanel,
} from '../helpers/add_visualization_helper';
import { createDemoPanel } from '../../../../server/common/helpers/custom_panels/sample_panels';
Copy link
Member

Choose a reason for hiding this comment

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

Should we move this to /common/constants/utils? Cause, the createDemoPanel function is defined in the server whereas it is being used in public component.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, +1, please relocate createDemoPanel fn

Copy link
Collaborator Author

@derek-ho derek-ho Apr 26, 2023

Choose a reason for hiding this comment

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

It's also being used in server here:

import { createDemoPanel } from '../../common/helpers/custom_panels/sample_panels';
@pjfitzgibbons @ps48 take a look and let me know what you think. I think the original use case is to use it in the server, I am just using it as well to create the SO panel

import {
addMultipleVisualizations,
addVisualizationPanel,
} from '../helpers/add_visualization_helper';
import { createDemoPanel } from '../../../../server/common/helpers/custom_panels/sample_panels';
Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, +1, please relocate createDemoPanel fn

Signed-off-by: Derek Ho <[email protected]>
Signed-off-by: Derek Ho <[email protected]>
@derek-ho derek-ho changed the base branch from main to 2.7 April 27, 2023 13:43
@derek-ho derek-ho changed the base branch from 2.7 to main April 27, 2023 13:43
@derek-ho derek-ho merged commit ce06ccb into opensearch-project:main Apr 27, 2023
@opensearch-trigger-bot
Copy link
Contributor

The backport to 2.x failed:

The process '/usr/bin/git' failed with exit code 1

To backport manually, run these commands in your terminal:

# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add .worktrees/backport-2.x 2.x
# Navigate to the new working tree
cd .worktrees/backport-2.x
# Create a new branch
git switch --create backport/backport-427-to-2.x
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 ce06ccb9fc4acd75d3ccbfbacab9db8a521de252
# Push it to GitHub
git push --set-upstream origin backport/backport-427-to-2.x
# Go back to the original working tree
cd ../..
# Delete the working tree
git worktree remove .worktrees/backport-2.x

Then, create a pull request where the base branch is 2.x and the compare/head branch is backport/backport-427-to-2.x.

@opensearch-trigger-bot
Copy link
Contributor

The backport to 2.7 failed:

The process '/usr/bin/git' failed with exit code 1

To backport manually, run these commands in your terminal:

# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add .worktrees/backport-2.7 2.7
# Navigate to the new working tree
cd .worktrees/backport-2.7
# Create a new branch
git switch --create backport/backport-427-to-2.7
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 ce06ccb9fc4acd75d3ccbfbacab9db8a521de252
# Push it to GitHub
git push --set-upstream origin backport/backport-427-to-2.7
# Go back to the original working tree
cd ../..
# Delete the working tree
git worktree remove .worktrees/backport-2.7

Then, create a pull request where the base branch is 2.7 and the compare/head branch is backport/backport-427-to-2.7.

derek-ho added a commit to derek-ho/dashboards-observability that referenced this pull request Apr 27, 2023
* change samples to saved object

Signed-off-by: Derek Ho <[email protected]>

* add sample to so

Signed-off-by: Derek Ho <[email protected]>

* fix saved object sample

Signed-off-by: Derek Ho <[email protected]>

* fix up

Signed-off-by: Derek Ho <[email protected]>

* add back toast

Signed-off-by: Derek Ho <[email protected]>

* revert file

Signed-off-by: Derek Ho <[email protected]>

* refactoring

Signed-off-by: Derek Ho <[email protected]>

* use constant

Signed-off-by: Derek Ho <[email protected]>

---------

Signed-off-by: Derek Ho <[email protected]>
(cherry picked from commit ce06ccb)
Signed-off-by: Derek Ho <[email protected]>
derek-ho added a commit that referenced this pull request May 1, 2023
* change samples to saved object



* add sample to so



* fix saved object sample



* fix up



* add back toast



* revert file



* refactoring



* use constant



---------


(cherry picked from commit ce06ccb)

Signed-off-by: Derek Ho <[email protected]>
ps48 added a commit that referenced this pull request May 24, 2023
* Fix for hover issue on docs of discover/explorer data grid (#329) (#358)

Signed-off-by: Eric Wei <[email protected]>
(cherry picked from commit 19f29e1)

Co-authored-by: Eric Wei <[email protected]>

* [2.7] Refactor Saved objects and add visualization embeddable (#341) (#353)

Signed-off-by: Joshua Li <[email protected]>
Co-authored-by: Eric Wei <[email protected]>

* Cohesion main to 2.7 (#382)

Signed-off-by: Eric Wei <[email protected]>
Signed-off-by: Joshua Li <[email protected]>
Signed-off-by: Peter Fitzgibbons <[email protected]>
Signed-off-by: Shenoy Pratik <[email protected]>
Signed-off-by: Derek Ho <[email protected]>
Co-authored-by: Derek Ho <[email protected]>
Co-authored-by: Eric Wei <[email protected]>
Co-authored-by: Joshua Li <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Shenoy Pratik <[email protected]>
Co-authored-by: Rupal Mahajan <[email protected]>

* Fix redirection to legacy event_analytics URL (#383) (#384)

Signed-off-by: Joshua Li <[email protected]>
(cherry picked from commit 23e09e5)

Co-authored-by: Joshua Li <[email protected]>

* [2.7] Redo pick from main to 2.7 (#385)

Signed-off-by: Peter Fitzgibbons <[email protected]>
Signed-off-by: Derek Ho <[email protected]>
Signed-off-by: Eric Wei <[email protected]>
Signed-off-by: Rupal Mahajan <[email protected]>
Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]>
Signed-off-by: Shenoy Pratik <[email protected]>
Signed-off-by: Joshua Li <[email protected]>
Co-authored-by: Eric Wei <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Derek Ho <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Rupal Mahajan <[email protected]>
Co-authored-by: Kavitha Conjeevaram Mohan <[email protected]>
Co-authored-by: Shenoy Pratik <[email protected]>

* Metrics analytics support for SOpanels  (#386) (#388)

Signed-off-by: Shenoy Pratik <[email protected]>
(cherry picked from commit 1ecd7ea)

Co-authored-by: Shenoy Pratik <[email protected]>

* change plugin ordering (#389) (#390)

Signed-off-by: Shenoy Pratik <[email protected]>
(cherry picked from commit a6c1ebc)

Co-authored-by: Shenoy Pratik <[email protected]>

* notebooks fix for cypress and minor bugs (#392) (#393)

* notebooks fix for cypress and minor bugs

Signed-off-by: Shenoy Pratik <[email protected]>

* remove comments

Signed-off-by: Shenoy Pratik <[email protected]>

---------

Signed-off-by: Shenoy Pratik <[email protected]>
(cherry picked from commit 3219f14)

* Update traces integration tests (#391) (#397)

* Update traces cypress tests

Signed-off-by: Rupal Mahajan <[email protected]>

* Update traces cypress test

Signed-off-by: Rupal Mahajan <[email protected]>

* Fix traces cypress tests

Signed-off-by: Rupal Mahajan <[email protected]>

* Fix filters test

Signed-off-by: Rupal Mahajan <[email protected]>

---------

Signed-off-by: Rupal Mahajan <[email protected]>
(cherry picked from commit b8f9ca3)

Co-authored-by: Rupal Mahajan <[email protected]>

* Fix redirection to legacy event_analytics URL from dashboards (#399) (#403)

Signed-off-by: Joshua Li <[email protected]>
(cherry picked from commit 05ef180)

Co-authored-by: Joshua Li <[email protected]>

* Adjust metrics top menu layout to avoid overflow (#398) (#402)

Signed-off-by: Joshua Li <[email protected]>
(cherry picked from commit 6309e63)

Co-authored-by: Joshua Li <[email protected]>

* Support duplicate visualization in dashboard (#400) (#405)

Signed-off-by: Joshua Li <[email protected]>
(cherry picked from commit 3d74bac)

Co-authored-by: Joshua Li <[email protected]>

* fix panel visualization preview, new viz workflow (#401) (#404)

Signed-off-by: Shenoy Pratik <[email protected]>
(cherry picked from commit 78347d9)

Co-authored-by: Shenoy Pratik <[email protected]>

* Fix saving multiple metrics to SOpanels (#407) (#408)

* fix saving multiple metrics to SOpanels

Signed-off-by: Shenoy Pratik <[email protected]>

* remove unused variable

Signed-off-by: Shenoy Pratik <[email protected]>

---------

Signed-off-by: Shenoy Pratik <[email protected]>
(cherry picked from commit a4cb6d1)

Co-authored-by: Shenoy Pratik <[email protected]>

* Panel table fixes (#406)

* Fix Panel View (legacy)
  - Duplicate
  - Rename
  - Delete

---------

Signed-off-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com>
Co-authored-by: Joshua Li <[email protected]>
Co-authored-by: Shenoy Pratik <[email protected]>

* Release Notes 2.7.0.0 (#394) (#412)

* Release Notes 2.7.0.0

Signed-off-by: Peter Fitzgibbons <[email protected]>

* Release Notes 2.7.0.0

Signed-off-by: Peter Fitzgibbons <[email protected]>

---------

Signed-off-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
(cherry picked from commit 969e639)

Co-authored-by: Peter Fitzgibbons <[email protected]>

* [Dashboard listing] update edit url and view url (#416) (#421)

* [Dashboard listing] update edit url and view url

Update to include BASE_URL if basePath is defined. Since create url is just
a direct nav and needs the basePath when passing props to the provider, the source
code no longer appends the basePath to keep these URLs consistent.

A feature should consolidate the basePaths in the dashboard listing service so that
createUrl also adds the base path. At which point we no longer need the base path.

Also removed some rogue URLs.

Core PR:
opensearch-project/OpenSearch-Dashboards#3922

---------

Signed-off-by: Kawika Avilla <[email protected]>
Signed-off-by: Joshua Li <[email protected]>
Co-authored-by: Joshua Li <[email protected]>
(cherry picked from commit f94c21e)

Co-authored-by: Kawika Avilla <[email protected]>

* Fix invalid time range in new panels (#419)

Signed-off-by: Joshua Li <[email protected]>

* open new tab for editing panel visualization (#430)

Signed-off-by: Eric Wei <[email protected]>

* Save new prometheus metrics to OSD saved objects (#428)

* Save new prometheus metrics to OSD saved objects

Signed-off-by: Joshua Li <[email protected]>

* Return if failed at saving metrics

Signed-off-by: Joshua Li <[email protected]>

---------

Signed-off-by: Joshua Li <[email protected]>

* fixes edit mode callback bug in Saved Object Panels (#420)

* fix for SOpanels editMode

Signed-off-by: Shenoy Pratik <[email protected]>

* rename edit button data-test-subj

Signed-off-by: Shenoy Pratik <[email protected]>

---------

Signed-off-by: Shenoy Pratik <[email protected]>

* Add Toasts to Observability Dashboards  (#435)

* Fixes
* Panel View (legacy)
  - Duplicate
  - Rename

Signed-off-by: Peter Fitzgibbons <[email protected]>

* Toasts use hook from useOpenSearchDashboards context provider

Signed-off-by: Peter Fitzgibbons <[email protected]>

* Testing for CustomPanel Toast

Signed-off-by: Peter Fitzgibbons <[email protected]>

* update catches from comments, minor code cleaning

Signed-off-by: Shenoy Pratik <[email protected]>

* update tests

Signed-off-by: Shenoy Pratik <[email protected]>

* remove unused redux slice

Signed-off-by: Shenoy Pratik <[email protected]>

* revert cypress changes

Signed-off-by: Shenoy Pratik <[email protected]>

* add toasts to SOflyout

Signed-off-by: Shenoy Pratik <[email protected]>

* fix messaging for multiple delete

Signed-off-by: Derek Ho <[email protected]>

* fix up toast and error handling for create and delete flows

Signed-off-by: Derek Ho <[email protected]>

* fix up clone

Signed-off-by: Derek Ho <[email protected]>

* fix rename in table

Signed-off-by: Derek Ho <[email protected]>

* fix rename in custom panel so view

Signed-off-by: Derek Ho <[email protected]>

* fix up panel toasts

Signed-off-by: Derek Ho <[email protected]>

* fix up for flyout

Signed-off-by: Derek Ho <[email protected]>

* code cleanup

Signed-off-by: Derek Ho <[email protected]>

* finish merge

Signed-off-by: Derek Ho <[email protected]>

* fix up PR comments

Signed-off-by: Derek Ho <[email protected]>

---------

Signed-off-by: Peter Fitzgibbons <[email protected]>
Signed-off-by: Shenoy Pratik <[email protected]>
Signed-off-by: Derek Ho <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Derek Ho <[email protected]>

* change samples to saved object (#427) (#436)

* change samples to saved object



* add sample to so



* fix saved object sample



* fix up



* add back toast



* revert file



* refactoring



* use constant



---------


(cherry picked from commit ce06ccb)

Signed-off-by: Derek Ho <[email protected]>

* Add Error handling on duplicate dashboard names (#441)

* Add check for existing panel name

Signed-off-by: Rupal Mahajan <[email protected]>

* Nit

Signed-off-by: Rupal Mahajan <[email protected]>

* Remove double await

Signed-off-by: Rupal Mahajan <[email protected]>

* Add rename name check test

Signed-off-by: Rupal Mahajan <[email protected]>

* Remove test

Signed-off-by: Rupal Mahajan <[email protected]>

* Add unique id to sample panel name

Signed-off-by: Rupal Mahajan <[email protected]>

---------

Signed-off-by: Rupal Mahajan <[email protected]>

* fix timestamp overriding issue (#449)

Signed-off-by: Eric Wei <[email protected]>

* Renaming custom panels to Observability Dashboards (#450)

* change custom panels -> observability dashboards

Signed-off-by: Eric Wei <[email protected]>

* rest of the panel renaming

Signed-off-by: Eric Wei <[email protected]>

---------

Signed-off-by: Eric Wei <[email protected]>

* Fix delete failure when no osd item selected (#452)

Signed-off-by: Joshua Li <[email protected]>
(cherry picked from commit a4a310e)

* Open log patterns by default (#444) (#459)

Signed-off-by: Joshua Li <[email protected]>
(cherry picked from commit 72970aa)

* diff merging between 2.x and 2.7

Signed-off-by: Shenoy Pratik <[email protected]>

* add service fields length check in app analytics

Signed-off-by: Shenoy Pratik <[email protected]>

---------

Signed-off-by: Joshua Li <[email protected]>
Signed-off-by: Eric Wei <[email protected]>
Signed-off-by: Peter Fitzgibbons <[email protected]>
Signed-off-by: Shenoy Pratik <[email protected]>
Signed-off-by: Derek Ho <[email protected]>
Signed-off-by: Rupal Mahajan <[email protected]>
Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]>
Co-authored-by: Joshua Li <[email protected]>
Co-authored-by: Eric Wei <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Derek Ho <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Rupal Mahajan <[email protected]>
Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com>
Co-authored-by: Kavitha Conjeevaram Mohan <[email protected]>
Co-authored-by: Kawika Avilla <[email protected]>
amsiglan pushed a commit to amsiglan/dashboards-observability that referenced this pull request Jun 7, 2024
* Fix for hover issue on docs of discover/explorer data grid (opensearch-project#329) (opensearch-project#358)

Signed-off-by: Eric Wei <[email protected]>
(cherry picked from commit 19f29e1)

Co-authored-by: Eric Wei <[email protected]>

* [2.7] Refactor Saved objects and add visualization embeddable (opensearch-project#341) (opensearch-project#353)

Signed-off-by: Joshua Li <[email protected]>
Co-authored-by: Eric Wei <[email protected]>

* Cohesion main to 2.7 (opensearch-project#382)

Signed-off-by: Eric Wei <[email protected]>
Signed-off-by: Joshua Li <[email protected]>
Signed-off-by: Peter Fitzgibbons <[email protected]>
Signed-off-by: Shenoy Pratik <[email protected]>
Signed-off-by: Derek Ho <[email protected]>
Co-authored-by: Derek Ho <[email protected]>
Co-authored-by: Eric Wei <[email protected]>
Co-authored-by: Joshua Li <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Shenoy Pratik <[email protected]>
Co-authored-by: Rupal Mahajan <[email protected]>

* Fix redirection to legacy event_analytics URL (opensearch-project#383) (opensearch-project#384)

Signed-off-by: Joshua Li <[email protected]>
(cherry picked from commit 23e09e5)

Co-authored-by: Joshua Li <[email protected]>

* [2.7] Redo pick from main to 2.7 (opensearch-project#385)

Signed-off-by: Peter Fitzgibbons <[email protected]>
Signed-off-by: Derek Ho <[email protected]>
Signed-off-by: Eric Wei <[email protected]>
Signed-off-by: Rupal Mahajan <[email protected]>
Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]>
Signed-off-by: Shenoy Pratik <[email protected]>
Signed-off-by: Joshua Li <[email protected]>
Co-authored-by: Eric Wei <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Derek Ho <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Rupal Mahajan <[email protected]>
Co-authored-by: Kavitha Conjeevaram Mohan <[email protected]>
Co-authored-by: Shenoy Pratik <[email protected]>

* Metrics analytics support for SOpanels  (opensearch-project#386) (opensearch-project#388)

Signed-off-by: Shenoy Pratik <[email protected]>
(cherry picked from commit 1ecd7ea)

Co-authored-by: Shenoy Pratik <[email protected]>

* change plugin ordering (opensearch-project#389) (opensearch-project#390)

Signed-off-by: Shenoy Pratik <[email protected]>
(cherry picked from commit a6c1ebc)

Co-authored-by: Shenoy Pratik <[email protected]>

* notebooks fix for cypress and minor bugs (opensearch-project#392) (opensearch-project#393)

* notebooks fix for cypress and minor bugs

Signed-off-by: Shenoy Pratik <[email protected]>

* remove comments

Signed-off-by: Shenoy Pratik <[email protected]>

---------

Signed-off-by: Shenoy Pratik <[email protected]>
(cherry picked from commit 3219f14)

* Update traces integration tests (opensearch-project#391) (opensearch-project#397)

* Update traces cypress tests

Signed-off-by: Rupal Mahajan <[email protected]>

* Update traces cypress test

Signed-off-by: Rupal Mahajan <[email protected]>

* Fix traces cypress tests

Signed-off-by: Rupal Mahajan <[email protected]>

* Fix filters test

Signed-off-by: Rupal Mahajan <[email protected]>

---------

Signed-off-by: Rupal Mahajan <[email protected]>
(cherry picked from commit b8f9ca3)

Co-authored-by: Rupal Mahajan <[email protected]>

* Fix redirection to legacy event_analytics URL from dashboards (opensearch-project#399) (opensearch-project#403)

Signed-off-by: Joshua Li <[email protected]>
(cherry picked from commit 05ef180)

Co-authored-by: Joshua Li <[email protected]>

* Adjust metrics top menu layout to avoid overflow (opensearch-project#398) (opensearch-project#402)

Signed-off-by: Joshua Li <[email protected]>
(cherry picked from commit 6309e63)

Co-authored-by: Joshua Li <[email protected]>

* Support duplicate visualization in dashboard (opensearch-project#400) (opensearch-project#405)

Signed-off-by: Joshua Li <[email protected]>
(cherry picked from commit 3d74bac)

Co-authored-by: Joshua Li <[email protected]>

* fix panel visualization preview, new viz workflow (opensearch-project#401) (opensearch-project#404)

Signed-off-by: Shenoy Pratik <[email protected]>
(cherry picked from commit 78347d9)

Co-authored-by: Shenoy Pratik <[email protected]>

* Fix saving multiple metrics to SOpanels (opensearch-project#407) (opensearch-project#408)

* fix saving multiple metrics to SOpanels

Signed-off-by: Shenoy Pratik <[email protected]>

* remove unused variable

Signed-off-by: Shenoy Pratik <[email protected]>

---------

Signed-off-by: Shenoy Pratik <[email protected]>
(cherry picked from commit a4cb6d1)

Co-authored-by: Shenoy Pratik <[email protected]>

* Panel table fixes (opensearch-project#406)

* Fix Panel View (legacy)
  - Duplicate
  - Rename
  - Delete

---------

Signed-off-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com>
Co-authored-by: Joshua Li <[email protected]>
Co-authored-by: Shenoy Pratik <[email protected]>

* Release Notes 2.7.0.0 (opensearch-project#394) (opensearch-project#412)

* Release Notes 2.7.0.0

Signed-off-by: Peter Fitzgibbons <[email protected]>

* Release Notes 2.7.0.0

Signed-off-by: Peter Fitzgibbons <[email protected]>

---------

Signed-off-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
(cherry picked from commit 969e639)

Co-authored-by: Peter Fitzgibbons <[email protected]>

* [Dashboard listing] update edit url and view url (opensearch-project#416) (opensearch-project#421)

* [Dashboard listing] update edit url and view url

Update to include BASE_URL if basePath is defined. Since create url is just
a direct nav and needs the basePath when passing props to the provider, the source
code no longer appends the basePath to keep these URLs consistent.

A feature should consolidate the basePaths in the dashboard listing service so that
createUrl also adds the base path. At which point we no longer need the base path.

Also removed some rogue URLs.

Core PR:
opensearch-project/OpenSearch-Dashboards#3922

---------

Signed-off-by: Kawika Avilla <[email protected]>
Signed-off-by: Joshua Li <[email protected]>
Co-authored-by: Joshua Li <[email protected]>
(cherry picked from commit f94c21e)

Co-authored-by: Kawika Avilla <[email protected]>

* Fix invalid time range in new panels (opensearch-project#419)

Signed-off-by: Joshua Li <[email protected]>

* open new tab for editing panel visualization (opensearch-project#430)

Signed-off-by: Eric Wei <[email protected]>

* Save new prometheus metrics to OSD saved objects (opensearch-project#428)

* Save new prometheus metrics to OSD saved objects

Signed-off-by: Joshua Li <[email protected]>

* Return if failed at saving metrics

Signed-off-by: Joshua Li <[email protected]>

---------

Signed-off-by: Joshua Li <[email protected]>

* fixes edit mode callback bug in Saved Object Panels (opensearch-project#420)

* fix for SOpanels editMode

Signed-off-by: Shenoy Pratik <[email protected]>

* rename edit button data-test-subj

Signed-off-by: Shenoy Pratik <[email protected]>

---------

Signed-off-by: Shenoy Pratik <[email protected]>

* Add Toasts to Observability Dashboards  (opensearch-project#435)

* Fixes
* Panel View (legacy)
  - Duplicate
  - Rename

Signed-off-by: Peter Fitzgibbons <[email protected]>

* Toasts use hook from useOpenSearchDashboards context provider

Signed-off-by: Peter Fitzgibbons <[email protected]>

* Testing for CustomPanel Toast

Signed-off-by: Peter Fitzgibbons <[email protected]>

* update catches from comments, minor code cleaning

Signed-off-by: Shenoy Pratik <[email protected]>

* update tests

Signed-off-by: Shenoy Pratik <[email protected]>

* remove unused redux slice

Signed-off-by: Shenoy Pratik <[email protected]>

* revert cypress changes

Signed-off-by: Shenoy Pratik <[email protected]>

* add toasts to SOflyout

Signed-off-by: Shenoy Pratik <[email protected]>

* fix messaging for multiple delete

Signed-off-by: Derek Ho <[email protected]>

* fix up toast and error handling for create and delete flows

Signed-off-by: Derek Ho <[email protected]>

* fix up clone

Signed-off-by: Derek Ho <[email protected]>

* fix rename in table

Signed-off-by: Derek Ho <[email protected]>

* fix rename in custom panel so view

Signed-off-by: Derek Ho <[email protected]>

* fix up panel toasts

Signed-off-by: Derek Ho <[email protected]>

* fix up for flyout

Signed-off-by: Derek Ho <[email protected]>

* code cleanup

Signed-off-by: Derek Ho <[email protected]>

* finish merge

Signed-off-by: Derek Ho <[email protected]>

* fix up PR comments

Signed-off-by: Derek Ho <[email protected]>

---------

Signed-off-by: Peter Fitzgibbons <[email protected]>
Signed-off-by: Shenoy Pratik <[email protected]>
Signed-off-by: Derek Ho <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Derek Ho <[email protected]>

* change samples to saved object (opensearch-project#427) (opensearch-project#436)

* change samples to saved object

* add sample to so

* fix saved object sample

* fix up

* add back toast

* revert file

* refactoring

* use constant

---------

(cherry picked from commit ce06ccb)

Signed-off-by: Derek Ho <[email protected]>

* Add Error handling on duplicate dashboard names (opensearch-project#441)

* Add check for existing panel name

Signed-off-by: Rupal Mahajan <[email protected]>

* Nit

Signed-off-by: Rupal Mahajan <[email protected]>

* Remove double await

Signed-off-by: Rupal Mahajan <[email protected]>

* Add rename name check test

Signed-off-by: Rupal Mahajan <[email protected]>

* Remove test

Signed-off-by: Rupal Mahajan <[email protected]>

* Add unique id to sample panel name

Signed-off-by: Rupal Mahajan <[email protected]>

---------

Signed-off-by: Rupal Mahajan <[email protected]>

* fix timestamp overriding issue (opensearch-project#449)

Signed-off-by: Eric Wei <[email protected]>

* Renaming custom panels to Observability Dashboards (opensearch-project#450)

* change custom panels -> observability dashboards

Signed-off-by: Eric Wei <[email protected]>

* rest of the panel renaming

Signed-off-by: Eric Wei <[email protected]>

---------

Signed-off-by: Eric Wei <[email protected]>

* Fix delete failure when no osd item selected (opensearch-project#452)

Signed-off-by: Joshua Li <[email protected]>
(cherry picked from commit a4a310e)

* Open log patterns by default (opensearch-project#444) (opensearch-project#459)

Signed-off-by: Joshua Li <[email protected]>
(cherry picked from commit 72970aa)

* diff merging between 2.x and 2.7

Signed-off-by: Shenoy Pratik <[email protected]>

* add service fields length check in app analytics

Signed-off-by: Shenoy Pratik <[email protected]>

---------

Signed-off-by: Joshua Li <[email protected]>
Signed-off-by: Eric Wei <[email protected]>
Signed-off-by: Peter Fitzgibbons <[email protected]>
Signed-off-by: Shenoy Pratik <[email protected]>
Signed-off-by: Derek Ho <[email protected]>
Signed-off-by: Rupal Mahajan <[email protected]>
Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]>
Co-authored-by: Joshua Li <[email protected]>
Co-authored-by: Eric Wei <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Derek Ho <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Rupal Mahajan <[email protected]>
Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com>
Co-authored-by: Kavitha Conjeevaram Mohan <[email protected]>
Co-authored-by: Kawika Avilla <[email protected]>
(cherry picked from commit 776d1ed)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants