Skip to content

Commit

Permalink
Remove injected ui app vars from Canvas
Browse files Browse the repository at this point in the history
  • Loading branch information
poffdeluxe committed Jan 28, 2020
1 parent 04ad88c commit a5411ac
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 9 deletions.
1 change: 0 additions & 1 deletion x-pack/legacy/plugins/canvas/__tests__/fixtures/kibana.js
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,6 @@ export class Plugin {
[this.props.name]: {},
elasticsearch: mockElasticsearch,
},
injectUiAppVars: noop,
config: () => ({
get: key => get(config, key),
has: key => has(config, key),
Expand Down
6 changes: 0 additions & 6 deletions x-pack/legacy/plugins/canvas/server/plugin.ts
Original file line number Diff line number Diff line change
Expand Up @@ -15,12 +15,6 @@ export class Plugin {

plugins.interpreter.register({ serverFunctions: functions });

core.injectUiAppVars('canvas', async () => {
return {
...plugins.kibana.injectedUiAppVars,
};
});

plugins.features.registerFeature({
id: 'canvas',
name: 'Canvas',
Expand Down
2 changes: 0 additions & 2 deletions x-pack/legacy/plugins/canvas/server/shim.ts
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@ export interface CoreSetup {
http: {
route: Legacy.Server['route'];
};
injectUiAppVars: Legacy.Server['injectUiAppVars'];
}

export interface PluginsSetup {
Expand Down Expand Up @@ -52,7 +51,6 @@ export async function createSetupShim(
...server.newPlatform.setup.core.http,
route: (...args) => server.route(...args),
},
injectUiAppVars: server.injectUiAppVars,
},
pluginsSetup: {
// @ts-ignore: New Platform not typed
Expand Down

0 comments on commit a5411ac

Please sign in to comment.