Skip to content

Commit

Permalink
search: remove search homepage usage
Browse files Browse the repository at this point in the history
The search home page project is on hold, removing usage of it from
enterprise_search plugin.
  • Loading branch information
TattdCodeMonkey committed Oct 3, 2024
1 parent e9dc77c commit 95bb0e0
Show file tree
Hide file tree
Showing 9 changed files with 34 additions and 259 deletions.
13 changes: 4 additions & 9 deletions x-pack/plugins/enterprise_search/kibana.jsonc
Original file line number Diff line number Diff line change
Expand Up @@ -7,17 +7,15 @@
"id": "enterpriseSearch",
"server": true,
"browser": true,
"configPath": [
"enterpriseSearch"
],
"configPath": ["enterpriseSearch"],
"requiredPlugins": [
"data",
"features",
"licensing",
"logsShared",
"logsDataAccess",
"esUiShared",
"navigation",
"navigation"
],
"optionalPlugins": [
"customIntegrations",
Expand All @@ -31,18 +29,15 @@
"guidedOnboarding",
"console",
"searchConnectors",
"searchHomepage",
"searchPlayground",
"searchInferenceEndpoints",
"embeddable",
"discover",
"charts",
"cloud",
"lens",
"share",
"share"
],
"requiredBundles": [
"kibanaReact"
]
"requiredBundles": ["kibanaReact"]
}
}

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,6 @@ export const mockKibanaProps: KibanaLogicProps = {
indexMappingComponent: () => {
return <></>;
},
isSearchHomepageEnabled: false,
isSidebarEnabled: true,
lens: {
EmbeddableComponent: jest.fn(),
Expand All @@ -86,7 +85,6 @@ export const mockKibanaProps: KibanaLogicProps = {
hasWebCrawler: true,
},
renderHeaderActions: jest.fn(),
searchHomepage: undefined,
searchPlayground: searchPlaygroundMock.createStart(),
security: securityMock.createStart(),
setBreadcrumbs: jest.fn(),
Expand Down
6 changes: 2 additions & 4 deletions x-pack/plugins/enterprise_search/public/navigation_tree.ts
Original file line number Diff line number Diff line change
Expand Up @@ -69,14 +69,12 @@ const euiItemTypeToNodeDefinition = ({

export const getNavigationTreeDefinition = ({
dynamicItems$,
isSearchHomepageEnabled,
}: {
dynamicItems$: Observable<DynamicSideNavItems>;
isSearchHomepageEnabled: boolean;
}): AddSolutionNavigationArg => {
return {
dataTestSubj: 'searchSideNav',
homePage: isSearchHomepageEnabled ? 'searchHomepage' : 'enterpriseSearch',
homePage: 'enterpriseSearch',
icon,
id: 'es',
navigationTree$: dynamicItems$.pipe(
Expand All @@ -88,7 +86,7 @@ export const getNavigationTreeDefinition = ({
breadcrumbStatus: 'hidden',
children: [
{
link: isSearchHomepageEnabled ? 'searchHomepage' : 'enterpriseSearch',
link: 'enterpriseSearch',
},
{
getIsActive: ({ pathNameSerialized, prepend }) => {
Expand Down
Loading

0 comments on commit 95bb0e0

Please sign in to comment.