From d2d622b3c386018630afcbab9e2eb999d3126b11 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?G=C3=A9ry=20Debongnie?= <ged@odoo.com>
Date: Fri, 1 Feb 2019 10:33:26 +0100
Subject: [PATCH] move some sideeffect code to makeEnvironment

---
 web/static/src/ts/env.ts  |  8 ++++++++
 web/static/src/ts/main.ts | 10 ----------
 2 files changed, 8 insertions(+), 10 deletions(-)

diff --git a/web/static/src/ts/env.ts b/web/static/src/ts/env.ts
index b3a5d4dc..8b791895 100644
--- a/web/static/src/ts/env.ts
+++ b/web/static/src/ts/env.ts
@@ -9,6 +9,8 @@ import {
 } from "./services/notifications";
 import { registry, Registry } from "./services/registry";
 import { IRouter, Router } from "./services/router";
+import { CRM } from "./widgets/crm";
+import { Discuss } from "./widgets/discuss";
 
 //------------------------------------------------------------------------------
 // Types
@@ -53,12 +55,18 @@ export interface Env extends WEnv {
  * this function will actually return the same environment.
  */
 export const makeEnvironment = memoize(function(): Env {
+  // main application registry
+  registry.add("action", "discuss", Discuss);
+  registry.add("action", "crm", CRM);
+
+  // services
   const qweb = new QWeb();
   const router = new Router();
   const ajax = new Ajax();
   const actionManager = new ActionManager(router, registry);
   const notifications = new NotificationManager();
 
+  // demo data
   const menus = [
     { title: "Discuss", actionID: 1 },
     { title: "CRM", actionID: 2 }
diff --git a/web/static/src/ts/main.ts b/web/static/src/ts/main.ts
index 85b3f845..251f5fae 100644
--- a/web/static/src/ts/main.ts
+++ b/web/static/src/ts/main.ts
@@ -3,16 +3,6 @@
 import { debounce } from "./core/utils";
 import { makeEnvironment } from "./env";
 import { Root } from "./root";
-import { registry } from "./services/registry";
-import { CRM } from "./widgets/crm";
-import { Discuss } from "./widgets/discuss";
-
-//------------------------------------------------------------------------------
-// Prepare application registry
-//------------------------------------------------------------------------------
-
-registry.add("action", "discuss", Discuss);
-registry.add("action", "crm", CRM);
 
 //------------------------------------------------------------------------------
 // Application bootstrapping