From 077e071c8974a78f7e2012bf64e62e0ee4ad7d31 Mon Sep 17 00:00:00 2001
From: "opensearch-trigger-bot[bot]"
 <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com>
Date: Tue, 10 Sep 2024 11:47:56 -0700
Subject: [PATCH] [Bug] Fix url redirection, add default render path (#387)
 (#392)

* bug fix for url redirection, support for modal popout



* change check to isNav



* fix formatting lint



* simplify router case for empty



---------



(cherry picked from commit 007eabdba528caa8897ffebd8d9738a6f2fd5aec)

Signed-off-by: Adam Tackett <tackadam@amazon.com>
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: Adam Tackett <tackadam@amazon.com>
---
 public/components/Main/main.tsx | 2 +-
 public/components/app.tsx       | 8 ++++----
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/public/components/Main/main.tsx b/public/components/Main/main.tsx
index b0aede8..3728710 100644
--- a/public/components/Main/main.tsx
+++ b/public/components/Main/main.tsx
@@ -291,7 +291,7 @@ export class Main extends React.Component<MainProps, MainState> {
       refreshTree: false,
       isAccelerationFlyoutOpened: false,
       isCallOutVisible: false,
-      cluster: 'Indexes',
+      cluster: props.urlDataSource ? 'Data source Connections' : 'Indexes',
       dataSourceOptions: [],
       selectedMDSDataConnectionId: this.props.dataSourceMDSId,
       mdsClusterName: '',
diff --git a/public/components/app.tsx b/public/components/app.tsx
index 5cf455f..2f90332 100644
--- a/public/components/app.tsx
+++ b/public/components/app.tsx
@@ -42,7 +42,7 @@ export const WorkbenchApp = ({
   setActionMenu,
 }: WorkbenchAppDeps) => {
   const isNavGroupEnabled = coreRefs?.chrome?.navGroup.getNavGroupEnabled();
-  const basePath = isNavGroupEnabled ? 'opensearch-query-workbench' : '';
+  const basePath = isNavGroupEnabled ? '/opensearch-query-workbench' : '';
 
   return (
     <HashRouter>
@@ -53,7 +53,7 @@ export const WorkbenchApp = ({
               <Switch>
                 <Route
                   exact
-                  path={`/${basePath}`}
+                  path={isNavGroupEnabled ? [`${basePath}`, `/`] : `/${basePath}`}
                   render={(props) => (
                     <Main
                       httpClient={http}
@@ -72,7 +72,7 @@ export const WorkbenchApp = ({
                 />
                 <Route
                   exact
-                  path={`/${basePath}/:dataSource`}
+                  path={`${basePath}/:dataSource`}
                   render={(props) => (
                     <Main
                       httpClient={http}
@@ -91,7 +91,7 @@ export const WorkbenchApp = ({
                 />
                 <Route
                   exact
-                  path={`/${basePath}/accelerate/:dataSource`}
+                  path={`${basePath}/accelerate/:dataSource`}
                   render={(props) => (
                     <Main
                       httpClient={http}