Skip to content

Commit

Permalink
Remove legacy ui-apps-mixin (elastic#76604)
Browse files Browse the repository at this point in the history
* remove ui_nav_links

* remove remove-ui-apps-mixin

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
  • Loading branch information
mshustov and elasticmachine authored Sep 4, 2020
1 parent c402e51 commit c547a81
Show file tree
Hide file tree
Showing 13 changed files with 6 additions and 1,015 deletions.
15 changes: 0 additions & 15 deletions src/legacy/server/kbn_server.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -50,10 +50,6 @@ import { HomeServerPluginSetup } from '../../plugins/home/server';
// lot of legacy code was assuming this type only had these two methods
export type KibanaConfig = Pick<LegacyConfig, 'get' | 'has'>;

export interface UiApp {
getId(): string;
}

// Extend the defaults with the plugins and server methods we need.
declare module 'hapi' {
interface PluginProperties {
Expand All @@ -66,13 +62,6 @@ declare module 'hapi' {
interface Server {
config: () => KibanaConfig;
savedObjects: SavedObjectsLegacyService;
injectUiAppVars: (pluginName: string, getAppVars: () => { [key: string]: any }) => void;
getHiddenUiAppById(appId: string): UiApp;
addScopedTutorialContextFactory: (
scopedTutorialContextFactory: (...args: any[]) => any
) => void;
getInjectedUiAppVars: (pluginName: string) => { [key: string]: any };
getUiNavLinks(): Array<{ _id: string }>;
logWithMetadata: (tags: string[], message: string, meta: Record<string, any>) => void;
newPlatform: KbnServer['newPlatform'];
}
Expand All @@ -82,10 +71,6 @@ declare module 'hapi' {
getBasePath(): string;
getUiSettingsService(): IUiSettingsClient;
}

interface ResponseToolkit {
renderAppWithDefaultConfig(app: UiApp): ResponseObject;
}
}

type KbnMixinFunc = (kbnServer: KbnServer, server: Server, config: any) => Promise<any> | void;
Expand Down
92 changes: 0 additions & 92 deletions src/legacy/ui/ui_apps/__snapshots__/ui_apps_mixin.test.js.snap

This file was deleted.

Loading

0 comments on commit c547a81

Please sign in to comment.