side navigation changes for notifications (#222) (#230)

* side navigation changes for notifications * fix jest tests * navLinks changes * fixed the issues * fixed the issues * addressed the comments * addressed the comments * Update public/plugin.ts --------- (cherry picked from commit 9e6066a) Signed-off-by: Riya Saxena <[email protected]> Signed-off-by: SuZhou-Joe <[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: SuZhou-Joe <[email protected]>
opensearch-project · Jul 30, 2024 · f04d2ea · f04d2ea
1 parent 306d149
commit f04d2ea
Show file tree

Hide file tree

Showing 3 changed files with 268 additions and 199 deletions.
diff --git a/public/application.tsx b/public/application.tsx
@@ -9,8 +9,6 @@ import { HashRouter as Router, Route } from 'react-router-dom';
 import { AppMountParameters, CoreStart } from '../../../src/core/public';
 import { CoreServicesContext } from './components/coreServices';
 import Main from './pages/Main';
-import { NotificationService } from './services';
-import { ServicesContext } from './services/services';
 import { DataSourceManagementPluginSetup } from '../../../src/plugins/data_source_management/public';
 import { AppPluginStartDependencies } from "./types";
 
@@ -19,6 +17,7 @@ export const renderApp = (
   params: AppMountParameters,
   dataSourceManagement: DataSourceManagementPluginSetup,
   pluginStartDependencies: AppPluginStartDependencies,
+  defaultRoute?: string,
 ) => {
 
   ReactDOM.render(
@@ -31,6 +30,7 @@ export const renderApp = (
                 multiDataSourceEnabled={!!pluginStartDependencies.dataSource}
                 dataSourceManagement={dataSourceManagement}
                 http={coreStart.http} // Pass http as a prop
+                defaultRoute={defaultRoute}
               />
             </CoreServicesContext.Provider>
         )}