[Bug] Fix url redirection, add default render path (#387)

* bug fix for url redirection, support for modal popout Signed-off-by: Adam Tackett <tackadam@amazon.com> * change check to isNav Signed-off-by: Adam Tackett <tackadam@amazon.com> * fix formatting lint Signed-off-by: Adam Tackett <tackadam@amazon.com> * simplify router case for empty Signed-off-by: Adam Tackett <tackadam@amazon.com> --------- Signed-off-by: Adam Tackett <tackadam@amazon.com> Co-authored-by: Adam Tackett <tackadam@amazon.com>
opensearch-project · Sep 6, 2024 · 007eabd · 007eabd
1 parent 8c545ae
commit 007eabd
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git 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
@@ -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}