Skip to content

Commit

Permalink
[navigation]feat: update category to flatten menus in analytics(all) …
Browse files Browse the repository at this point in the history
…use case (#448) (#456)

* feat: update category



* Bump upload artifact version



---------




(cherry picked from commit f50ed4c)

Signed-off-by: SuZhou-Joe <[email protected]>
Signed-off-by: Derek Ho <[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>
Co-authored-by: Derek Ho <[email protected]>
  • Loading branch information
3 people authored Sep 30, 2024
1 parent 9b7e78f commit 49d813c
Show file tree
Hide file tree
Showing 4 changed files with 15 additions and 8 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/cypress-e2e-reporting-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -154,14 +154,14 @@ jobs:
working-directory: OpenSearch-Dashboards/plugins/${{ env.PLUGIN_NAME }}

- name: Capture failure screenshots
uses: actions/upload-artifact@v1
uses: actions/upload-artifact@v4.4.0
if: failure()
with:
name: cypress-screenshots-${{ matrix.os }}
path: OpenSearch-Dashboards/plugins/${{ env.PLUGIN_NAME }}/.cypress/screenshots

- name: Capture test video
uses: actions/upload-artifact@v1
uses: actions/upload-artifact@v4.4.0
if: failure()
with:
name: cypress-videos-${{ matrix.os }}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ jobs:
whoami && yarn build && mv -v ./build/*.zip ./build/${{ env.ARTIFACT_NAME }}-${{ env.OPENSEARCH_PLUGIN_VERSION }}.zip"
- name: Upload Artifact For Linux
uses: actions/upload-artifact@v1
uses: actions/upload-artifact@v4.4.0
with:
name: dashboards-reports-linux
path: ./OpenSearch-Dashboards/plugins/${{ env.PLUGIN_NAME }}/build/${{ env.ARTIFACT_NAME }}-${{ env.OPENSEARCH_PLUGIN_VERSION }}.zip
Expand Down Expand Up @@ -128,7 +128,7 @@ jobs:
mv ./build/*.zip ./build/${{ env.ARTIFACT_NAME }}-${{ env.OPENSEARCH_PLUGIN_VERSION }}.zip
- name: Upload Artifact For Windows
uses: actions/upload-artifact@v1
uses: actions/upload-artifact@v4.4.0
with:
name: dashboards-reports-windows
path: ./build/${{ env.ARTIFACT_NAME }}-${{ env.OPENSEARCH_PLUGIN_VERSION }}.zip
Expand Down Expand Up @@ -185,7 +185,7 @@ jobs:
mv ./build/*.zip ./build/${{ env.ARTIFACT_NAME }}-${{ env.OPENSEARCH_PLUGIN_VERSION }}.zip
- name: Upload Artifact For MacOS
uses: actions/upload-artifact@v1
uses: actions/upload-artifact@v4.4.0
with:
name: dashboards-reports-macosx
path: ./build/${{ env.ARTIFACT_NAME }}-${{ env.OPENSEARCH_PLUGIN_VERSION }}.zip
4 changes: 2 additions & 2 deletions .github/workflows/ftr-e2e-reporting-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -162,14 +162,14 @@ jobs:
working-directory: opensearch-dashboards-functional-test

- name: Capture failure screenshots
uses: actions/upload-artifact@v1
uses: actions/upload-artifact@v4.4.0
if: failure()
with:
name: cypress-screenshots-${{ matrix.os }}
path: opensearch-dashboards-functional-test/cypress/screenshots

- name: Capture test video
uses: actions/upload-artifact@v1
uses: actions/upload-artifact@v4.4.0
if: failure()
with:
name: cypress-videos-${{ matrix.os }}
Expand Down
9 changes: 8 additions & 1 deletion public/plugin_nav.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,14 @@ export function registerAllPluginNavGroups(core: CoreSetup<ReportsDashboardsPlug
core.chrome.navGroup.addNavLinksToGroup(DEFAULT_NAV_GROUPS.search, [
{
id: PLUGIN_ID,
category: DEFAULT_APP_CATEGORIES.analyzeSearch,
category: DEFAULT_APP_CATEGORIES.visualizeAndReport,
order: 300,
},
]);
core.chrome.navGroup.addNavLinksToGroup(DEFAULT_NAV_GROUPS.all, [
{
id: PLUGIN_ID,
category: DEFAULT_APP_CATEGORIES.visualizeAndReport,
order: 300,
},
]);
Expand Down

0 comments on commit 49d813c

Please sign in to comment.