Skip to content

Commit

Permalink
Cleanup and adding some types
Browse files Browse the repository at this point in the history
  • Loading branch information
poffdeluxe committed May 6, 2020
1 parent 23b4e68 commit 18c82f2
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 6 deletions.
5 changes: 3 additions & 2 deletions x-pack/plugins/canvas/public/application.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,9 @@ import ReactDOM from 'react-dom';
import { I18nProvider } from '@kbn/i18n/react';
import { i18n } from '@kbn/i18n';
import { Provider } from 'react-redux';
import { Observable } from 'rxjs';

import { AppMountParameters, CoreStart, CoreSetup } from 'kibana/public';
import { AppMountParameters, CoreStart, CoreSetup, AppUpdater } from 'kibana/public';

import { CanvasStartDeps, CanvasSetupDeps } from './plugin';
// @ts-ignore Untyped local
Expand Down Expand Up @@ -89,7 +90,7 @@ export const initializeCanvas = async (
setupPlugins: CanvasSetupDeps,
startPlugins: CanvasStartDeps,
registries: SetupRegistries,
navUpdater: any
navUpdater: Observable<AppUpdater>
) => {
startServices(coreSetup, coreStart, setupPlugins, startPlugins, navUpdater);

Expand Down
7 changes: 4 additions & 3 deletions x-pack/plugins/canvas/public/services/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,8 @@
* you may not use this file except in compliance with the Elastic License.
*/

import { CoreSetup, CoreStart } from '../../../../../src/core/public';
import { Observable } from 'rxjs';
import { CoreSetup, CoreStart, AppUpdater } from '../../../../../src/core/public';
import { CanvasSetupDeps, CanvasStartDeps } from '../plugin';
import { notifyServiceFactory } from './notify';
import { platformServiceFactory } from './platform';
Expand All @@ -15,7 +16,7 @@ export type CanvasServiceFactory<Service> = (
coreStart: CoreStart,
canvasSetupPlugins: CanvasSetupDeps,
canvasStartPlugins: CanvasStartDeps,
appUpdater: any
appUpdater: Observable<AppUpdater>
) => Service;

class CanvasServiceProvider<Service> {
Expand All @@ -31,7 +32,7 @@ class CanvasServiceProvider<Service> {
coreStart: CoreStart,
canvasSetupPlugins: CanvasSetupDeps,
canvasStartPlugins: CanvasStartDeps,
appUpdater: any
appUpdater: Observable<AppUpdater>
) {
this.service = this.factory(
coreSetup,
Expand Down
1 change: 0 additions & 1 deletion x-pack/plugins/canvas/public/services/nav_link.ts
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@
*/

import { CanvasServiceFactory } from '.';
import { CoreStart, CoreSetup, CanvasSetupDeps, CanvasStartDeps } from '../plugin';
import { SESSIONSTORAGE_LASTPATH } from '../../common/lib/constants';

interface NavLinkService {
Expand Down

0 comments on commit 18c82f2

Please sign in to comment.