From 74c3a117701efeac9cc899cbb6d3c4d5f8c5047f Mon Sep 17 00:00:00 2001
From: Stacey Gammon <gammon@elastic.co>
Date: Mon, 10 Feb 2020 13:13:50 -0500
Subject: [PATCH] Move action registration out of AppMountContext fn

---
 .../ui_actions_explorer/public/plugin.tsx     | 41 ++++++++++---------
 1 file changed, 21 insertions(+), 20 deletions(-)

diff --git a/examples/ui_actions_explorer/public/plugin.tsx b/examples/ui_actions_explorer/public/plugin.tsx
index f2b11b12f4a14..ab374f15aab74 100644
--- a/examples/ui_actions_explorer/public/plugin.tsx
+++ b/examples/ui_actions_explorer/public/plugin.tsx
@@ -17,7 +17,7 @@
  * under the License.
  */
 
-import { Plugin, CoreSetup, AppMountParameters } from 'kibana/public';
+import { Plugin, CoreSetup, AppMountParameters, CoreStart } from 'kibana/public';
 import { UiActionsStart, UiActionsSetup } from 'src/plugins/ui_actions/public';
 import { ISearchAppMountContext } from '../../../src/plugins/data/public';
 import {
@@ -66,30 +66,25 @@ export class UiActionsExplorerPlugin implements Plugin<void, void, {}, StartDeps
       id: USER_TRIGGER,
       actionIds: [],
     });
+    deps.uiActions.registerAction(lookUpWeatherAction);
+    deps.uiActions.registerAction(viewInMapsAction);
+    deps.uiActions.registerAction(makePhoneCallAction);
+    deps.uiActions.registerAction(showcasePluggability);
+
+    // What's missing here is type analysis to ensure the context emitted by the trigger
+    // is the same context that the action requires.
+    deps.uiActions.attachAction(COUNTRY_TRIGGER, VIEW_IN_MAPS_ACTION);
+    deps.uiActions.attachAction(COUNTRY_TRIGGER, TRAVEL_GUIDE_ACTION);
+    deps.uiActions.attachAction(COUNTRY_TRIGGER, SHOWCASE_PLUGGABILITY_ACTION);
+    deps.uiActions.attachAction(PHONE_TRIGGER, CALL_PHONE_NUMBER_ACTION);
+    deps.uiActions.attachAction(PHONE_TRIGGER, SHOWCASE_PLUGGABILITY_ACTION);
+    deps.uiActions.attachAction(USER_TRIGGER, SHOWCASE_PLUGGABILITY_ACTION);
 
     core.application.register({
       id: 'uiActionsExplorer',
       title: 'Ui Actions Explorer',
       async mount(params: AppMountParameters) {
         const [coreStart, depsStart] = await core.getStartServices();
-        deps.uiActions.registerAction(createPhoneUserAction(depsStart.uiActions));
-        deps.uiActions.registerAction(lookUpWeatherAction);
-        deps.uiActions.registerAction(viewInMapsAction);
-        deps.uiActions.registerAction(createEditUserAction(coreStart.overlays.openModal));
-        deps.uiActions.registerAction(makePhoneCallAction);
-        deps.uiActions.registerAction(showcasePluggability);
-
-        // What's missing here is type analysis to ensure the context emitted by the trigger
-        // is the same context that the action requires.
-        deps.uiActions.attachAction(PHONE_TRIGGER, CALL_PHONE_NUMBER_ACTION);
-        deps.uiActions.attachAction(PHONE_TRIGGER, SHOWCASE_PLUGGABILITY_ACTION);
-        deps.uiActions.attachAction(COUNTRY_TRIGGER, VIEW_IN_MAPS_ACTION);
-        deps.uiActions.attachAction(COUNTRY_TRIGGER, TRAVEL_GUIDE_ACTION);
-        deps.uiActions.attachAction(COUNTRY_TRIGGER, SHOWCASE_PLUGGABILITY_ACTION);
-        deps.uiActions.attachAction(USER_TRIGGER, PHONE_USER_ACTION);
-        deps.uiActions.attachAction(USER_TRIGGER, EDIT_USER_ACTION);
-        deps.uiActions.attachAction(USER_TRIGGER, SHOWCASE_PLUGGABILITY_ACTION);
-
         const { renderApp } = await import('./app');
         return renderApp(
           { uiActionsApi: depsStart.uiActions, openModal: coreStart.overlays.openModal },
@@ -99,6 +94,12 @@ export class UiActionsExplorerPlugin implements Plugin<void, void, {}, StartDeps
     });
   }
 
-  public start() {}
+  public start(core: CoreStart, deps: StartDeps) {
+    deps.uiActions.registerAction(createPhoneUserAction(deps.uiActions));
+    deps.uiActions.registerAction(createEditUserAction(core.overlays.openModal));
+    deps.uiActions.attachAction(USER_TRIGGER, PHONE_USER_ACTION);
+    deps.uiActions.attachAction(USER_TRIGGER, EDIT_USER_ACTION);
+  }
+
   public stop() {}
 }