Skip to content

Commit

Permalink
Remove injected ui app vars from Canvas (#56190)
Browse files Browse the repository at this point in the history
Co-authored-by: Elastic Machine <[email protected]>
  • Loading branch information
poffdeluxe and elasticmachine authored Feb 11, 2020
1 parent c0497ce commit d0dfacd
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 @@ -12,12 +12,6 @@ export class Plugin {
public setup(core: CoreSetup, plugins: PluginsSetup) {
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 d0dfacd

Please sign in to comment.