orc-shared 1.2.0-dev.1 → 1.2.0-dev.13

package/dist/utils/propertyHelper.js

@@ -1,9 +1,9 @@
 "use strict";
 
 exports.__esModule = true;
-exports.getPropertyOrDefaultFromMap = getPropertyOrDefaultFromMap;
 exports.getPropertyOrDefault = getPropertyOrDefault;
-exports.isObjectContainsPropertyWithAnyValue = exports.isObjectContainsPropertyWithValue = void 0;
+exports.getPropertyOrDefaultFromMap = getPropertyOrDefaultFromMap;
+exports.isObjectContainsPropertyWithValue = exports.isObjectContainsPropertyWithAnyValue = void 0;
 
 (function () {
   var enterModule = typeof reactHotLoaderGlobal !== 'undefined' ? reactHotLoaderGlobal.enterModule : undefined;

package/dist/utils/propertyValidator.js

@@ -1,7 +1,7 @@
 "use strict";
 
 exports.__esModule = true;
-exports.isReactComponent = exports.isStringNullOrWhitespace = exports.isObject = exports.isString = void 0;
+exports.isStringNullOrWhitespace = exports.isString = exports.isReactComponent = exports.isObject = void 0;
 
 var _react = _interopRequireDefault(require("react"));
 

package/dist/utils/setTranslation.js

@@ -15,6 +15,12 @@ var _flatten = require("./flatten");
   enterModule && enterModule(module);
 })();
 
+function _createForOfIteratorHelperLoose(o, allowArrayLike) { var it = typeof Symbol !== "undefined" && o[Symbol.iterator] || o["@@iterator"]; if (it) return (it = it.call(o)).next.bind(it); if (Array.isArray(o) || (it = _unsupportedIterableToArray(o)) || allowArrayLike && o && typeof o.length === "number") { if (it) o = it; var i = 0; return function () { if (i >= o.length) return { done: true }; return { done: false, value: o[i++] }; }; } throw new TypeError("Invalid attempt to iterate non-iterable instance.\nIn order to be iterable, non-array objects must have a [Symbol.iterator]() method."); }
+
+function _unsupportedIterableToArray(o, minLen) { if (!o) return; if (typeof o === "string") return _arrayLikeToArray(o, minLen); var n = Object.prototype.toString.call(o).slice(8, -1); if (n === "Object" && o.constructor) n = o.constructor.name; if (n === "Map" || n === "Set") return Array.from(o); if (n === "Arguments" || /^(?:Ui|I)nt(?:8|16|32)(?:Clamped)?Array$/.test(n)) return _arrayLikeToArray(o, minLen); }
+
+function _arrayLikeToArray(arr, len) { if (len == null || len > arr.length) len = arr.length; for (var i = 0, arr2 = new Array(len); i < len; i++) { arr2[i] = arr[i]; } return arr2; }
+
 var __signature__ = typeof reactHotLoaderGlobal !== 'undefined' ? reactHotLoaderGlobal.default.signature : function (a) {
   return a;
 };
@@ -30,7 +36,27 @@ var setTranslation = function setTranslation(locale, obj) {
   }
 
   if (!obj || !obj.getIn((0, _flatten.flatten)([field]))) return obj;
-  var value = obj.getIn((0, _flatten.flatten)([field, locale])) || obj.getIn((0, _flatten.flatten)([field])).filter(function (i) {
+  var localeValue = obj.getIn((0, _flatten.flatten)([field, locale]));
+
+  if (!localeValue && locale.includes("-")) {
+    var fieldValue = obj.getIn((0, _flatten.flatten)([field]));
+    var fieldKeys = fieldValue.keys();
+    var language = locale.substring(0, locale.indexOf("-"));
+
+    for (var _iterator = _createForOfIteratorHelperLoose(fieldKeys), _step; !(_step = _iterator()).done;) {
+      var key = _step.value;
+
+      if (key !== locale && key.startsWith(language)) {
+        localeValue = obj.getIn((0, _flatten.flatten)([field, key]));
+
+        if (localeValue) {
+          break;
+        }
+      }
+    }
+  }
+
+  var value = localeValue || obj.getIn((0, _flatten.flatten)([field])).filter(function (i) {
     return i;
   }).first() || "";
   return obj.setIn((0, _flatten.flatten)([field]), value);

package/dist/utils/setTranslationWithFallback.js

@@ -1,7 +1,7 @@
 "use strict";
 
 exports.__esModule = true;
-exports.setTranslationWithFallbackField = exports.setTranslationWithFallbackValue = void 0;
+exports.setTranslationWithFallbackValue = exports.setTranslationWithFallbackField = void 0;
 
 var _flatten = require("./flatten");
 
@@ -15,6 +15,12 @@ var _flatten = require("./flatten");
   enterModule && enterModule(module);
 })();
 
+function _createForOfIteratorHelperLoose(o, allowArrayLike) { var it = typeof Symbol !== "undefined" && o[Symbol.iterator] || o["@@iterator"]; if (it) return (it = it.call(o)).next.bind(it); if (Array.isArray(o) || (it = _unsupportedIterableToArray(o)) || allowArrayLike && o && typeof o.length === "number") { if (it) o = it; var i = 0; return function () { if (i >= o.length) return { done: true }; return { done: false, value: o[i++] }; }; } throw new TypeError("Invalid attempt to iterate non-iterable instance.\nIn order to be iterable, non-array objects must have a [Symbol.iterator]() method."); }
+
+function _unsupportedIterableToArray(o, minLen) { if (!o) return; if (typeof o === "string") return _arrayLikeToArray(o, minLen); var n = Object.prototype.toString.call(o).slice(8, -1); if (n === "Object" && o.constructor) n = o.constructor.name; if (n === "Map" || n === "Set") return Array.from(o); if (n === "Arguments" || /^(?:Ui|I)nt(?:8|16|32)(?:Clamped)?Array$/.test(n)) return _arrayLikeToArray(o, minLen); }
+
+function _arrayLikeToArray(arr, len) { if (len == null || len > arr.length) len = arr.length; for (var i = 0, arr2 = new Array(len); i < len; i++) { arr2[i] = arr[i]; } return arr2; }
+
 var __signature__ = typeof reactHotLoaderGlobal !== 'undefined' ? reactHotLoaderGlobal.default.signature : function (a) {
   return a;
 };
@@ -30,7 +36,30 @@ var setTranslationWithFallbackValue = function setTranslationWithFallbackValue(l
     field[_key - 3] = arguments[_key];
   }
 
-  var value = obj.getIn((0, _flatten.flatten)([field, locale])) || fallbackValue || "";
+  var localeValue = obj.getIn((0, _flatten.flatten)([field, locale]));
+
+  if (!localeValue && locale.includes("-")) {
+    var fieldValue = obj.getIn((0, _flatten.flatten)([field]));
+
+    if (fieldValue) {
+      var fieldKeys = fieldValue.keys();
+      var language = locale.substring(0, locale.indexOf("-"));
+
+      for (var _iterator = _createForOfIteratorHelperLoose(fieldKeys), _step; !(_step = _iterator()).done;) {
+        var key = _step.value;
+
+        if (key !== locale && key.startsWith(language)) {
+          localeValue = obj.getIn((0, _flatten.flatten)([field, key]));
+
+          if (localeValue) {
+            break;
+          }
+        }
+      }
+    }
+  }
+
+  var value = localeValue || fallbackValue || "";
   return obj.setIn((0, _flatten.flatten)([field]), value);
 };
 

package/dist/utils/testUtils.js

@@ -1,8 +1,9 @@
 "use strict";
 
 exports.__esModule = true;
+exports.createMuiTheme = exports.TestWrapper = exports.PropStruct = void 0;
 exports.extractMessages = extractMessages;
-exports.TestWrapper = exports.generateClassName = exports.createMuiTheme = exports.ignoreConsoleError = exports.PropStruct = exports.firstItemComparator = exports.getStyledClassSelector = exports.getClassSelector = exports.getClassName = exports.getElmClasses = exports.spyOnConsole = void 0;
+exports.spyOnConsole = exports.ignoreConsoleError = exports.getStyledClassSelector = exports.getElmClasses = exports.getClassSelector = exports.getClassName = exports.generateClassName = exports.firstItemComparator = void 0;
 
 var _react = _interopRequireDefault(require("react"));
 

package/dist/utils/unwrapImmutable.js

@@ -1,7 +1,7 @@
 "use strict";
 
 exports.__esModule = true;
-exports.default = exports.unwrapImmutable = void 0;
+exports.unwrapImmutable = exports.default = void 0;
 
 var _immutable = _interopRequireDefault(require("immutable4"));
 

package/dist/utils/urlHelper.js

@@ -1,7 +1,7 @@
 "use strict";
 
 exports.__esModule = true;
-exports.resolveEntityId = exports.tryGetNewEntityIdKey = exports.NEW_ENTITY_URL_KEY = exports.getValueFromUrlByKey = void 0;
+exports.tryGetNewEntityIdKey = exports.resolveEntityId = exports.getValueFromUrlByKey = exports.NEW_ENTITY_URL_KEY = void 0;
 
 (function () {
   var enterModule = typeof reactHotLoaderGlobal !== 'undefined' ? reactHotLoaderGlobal.enterModule : undefined;

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "orc-shared",
-	"version": "1.2.0-dev.1",
+	"version": "1.2.0-dev.13",
 	"description": "Shared code for Orckestra applications",
 	"main": "./src/index.js",
 	"exports": {
@@ -47,11 +47,9 @@
 			"cultureIso": "fr-CA"
 		}
 	],
-	"devDependencies": {
-		"orc-scripts": "1.3.0-dev.1"
-	},
+	"devDependencies": {},
 	"peerDependencies": {
-		"orc-scripts": "1.3.0-dev.1"
+		"orc-scripts": "1.3.0-dev.2"
 	},
 	"sideEffects": false,
 	"lint-staged": {
@@ -69,5 +67,8 @@
 		"hooks": {
 			"pre-commit": "lint-staged"
 		}
+	},
+	"dependencies": {
+		"orc-scripts": "1.3.0-dev.2"
 	}
 }

package/src/actions/modules.js

@@ -4,3 +4,33 @@ export const setModulesStructure = modules => ({
 	type: SET_MODULES_STRUCTURE,
 	payload: modules,
 });
+
+export const SET_MODULE_AS_VISIBLE = "SET_MODULE_AS_VISIBLE";
+
+export const setModuleAsVisible = module => ({
+	type: SET_MODULE_AS_VISIBLE,
+	payload: module,
+});
+
+export const INITIALIZE_FIRST_MODULE_SCOPE = "INITIALIZE_FIRST_MODULE_SCOPE";
+
+export const initializeFirstModuleScope = scope => ({
+	type: INITIALIZE_FIRST_MODULE_SCOPE,
+	payload: scope,
+});
+
+export const SET_NEW_SCOPE_AND_MODULE_NAME = "SET_NEW_SCOPE_AND_MODULE_NAME";
+
+export const setNewScopeAndModuleName = (scope, moduleName) => ({
+	type: SET_NEW_SCOPE_AND_MODULE_NAME,
+	payload: {
+		scope,
+		moduleName,
+	},
+});
+
+export const SET_ROUTING_PERFORMED = "SET_ROUTING_PERFORMED";
+
+export const setRoutingPerformed = () => ({
+	type: SET_ROUTING_PERFORMED,
+});

package/src/actions/modules.test.js

@@ -1,4 +1,15 @@
-import { SET_MODULES_STRUCTURE, setModulesStructure } from "./modules";
+import {
+	INITIALIZE_FIRST_MODULE_SCOPE,
+	initializeFirstModuleScope,
+	SET_MODULE_AS_VISIBLE,
+	SET_MODULES_STRUCTURE,
+	SET_NEW_SCOPE_AND_MODULE_NAME,
+	SET_ROUTING_PERFORMED,
+	setModuleAsVisible,
+	setModulesStructure,
+	setNewScopeAndModuleName,
+	setRoutingPerformed,
+} from "./modules";
 
 describe("setModulesStructure", () => {
 	it("creates an action object", () => {
@@ -8,3 +19,41 @@ describe("setModulesStructure", () => {
 		});
 	});
 });
+
+describe("initializeFirstModuleScope", () => {
+	it("creates an action object", () => {
+		expect(initializeFirstModuleScope, "when called with", ["module_1"], "to equal", {
+			type: INITIALIZE_FIRST_MODULE_SCOPE,
+			payload: "module_1",
+		});
+	});
+});
+
+describe("setModuleAsVisible", () => {
+	it("creates an action object", () => {
+		expect(setModuleAsVisible, "when called with", ["module_A"], "to equal", {
+			type: SET_MODULE_AS_VISIBLE,
+			payload: "module_A",
+		});
+	});
+});
+
+describe("setNewScopeAndModuleName", () => {
+	it("creates an action object", () => {
+		expect(setNewScopeAndModuleName, "when called with", ["matrixScope", "moduleNeo"], "to equal", {
+			type: SET_NEW_SCOPE_AND_MODULE_NAME,
+			payload: {
+				scope: "matrixScope",
+				moduleName: "moduleNeo",
+			},
+		});
+	});
+});
+
+describe("setRoutingPerformed", () => {
+	it("creates an action object", () => {
+		expect(setRoutingPerformed, "when called", "to equal", {
+			type: SET_ROUTING_PERFORMED,
+		});
+	});
+});

package/src/actions/scopes.js

@@ -1,26 +1,52 @@
 import { makeActionTypes } from "./makeApiAction";
 import makeOrcApiAction from "./makeOrcApiAction";
-import { getUserScopeRequest, getUserScopeTreeRequest } from "./requestsApi";
+import { getApplicationModules, getUserScopeRequest, getUserScopeTreeRequest } from "./requestsApi";
+import { overtureModule } from "../constants";
 
-export const validateOvertureModule = () => {
-	if (!OVERTURE_MODULE) {
-		throw new Error('"overtureModule.name" is missing in the configuration.');
+export const validateOvertureApplication = () => {
+	if (!OVERTURE_APPLICATION) {
+		throw new Error('"overtureApplication.name" is missing in the configuration.');
 	}
 };
 
-validateOvertureModule();
+validateOvertureApplication();
+
+export const GET_APPLICATION_MODULES = "GET_APPLICATION_MODULES";
+
+export const [GET_APPLICATION_MODULES_REQUEST, GET_APPLICATION_MODULES_SUCCESS, GET_APPLICATION_MODULES_FAILURE] =
+	makeActionTypes(GET_APPLICATION_MODULES);
+
+export const getAppModules = () =>
+	makeOrcApiAction(GET_APPLICATION_MODULES, getApplicationModules.buildUrl(OVERTURE_APPLICATION));
 
 export const GET_SCOPES = "GET_SCOPES";
 
 export const [GET_SCOPES_REQUEST, GET_SCOPES_SUCCESS, GET_SCOPES_FAILURE] = makeActionTypes(GET_SCOPES);
 
-export const getScopes = () => makeOrcApiAction(GET_SCOPES, getUserScopeTreeRequest.buildUrl(OVERTURE_MODULE, {}));
+export const getScopes = module =>
+	makeOrcApiAction(
+		GET_SCOPES,
+		getUserScopeTreeRequest.buildUrl(module === overtureModule.System ? overtureModule.Orders : module, {}),
+		getUserScopeTreeRequest.verb,
+		{
+			bailout: false,
+			meta: { module },
+		},
+	);
 
 export const GET_MY_SCOPE = "GET_MY_SCOPE";
 
 export const [GET_MY_SCOPE_REQUEST, GET_MY_SCOPE_SUCCESS, GET_MY_SCOPE_FAILURE] = makeActionTypes(GET_MY_SCOPE);
 
-export const getDefaultScope = () => makeOrcApiAction(GET_MY_SCOPE, getUserScopeRequest.buildUrl(OVERTURE_MODULE));
+export const getDefaultScope = module =>
+	makeOrcApiAction(
+		GET_MY_SCOPE,
+		getUserScopeRequest.buildUrl(module === overtureModule.System ? overtureModule.Orders : module),
+		getUserScopeRequest.verb,
+		{
+			bailout: false,
+		},
+	);
 
 export const APPLICATION_SCOPE_HAS_CHANGED = "APPLICATION_SCOPE_HAS_CHANGED";
 

package/src/actions/scopes.test.js

@@ -1,16 +1,17 @@
 import { RSAA } from "redux-api-middleware";
 import {
-	GET_SCOPES_REQUEST,
-	GET_SCOPES_SUCCESS,
-	GET_SCOPES_FAILURE,
 	getScopes,
 	GET_MY_SCOPE_REQUEST,
 	GET_MY_SCOPE_SUCCESS,
 	GET_MY_SCOPE_FAILURE,
 	getDefaultScope,
-	validateOvertureModule,
+	validateOvertureApplication,
 	APPLICATION_SCOPE_HAS_CHANGED,
 	applicationScopeHasChanged,
+	getAppModules,
+	GET_APPLICATION_MODULES_REQUEST,
+	GET_APPLICATION_MODULES_SUCCESS,
+	GET_APPLICATION_MODULES_FAILURE,
 } from "./scopes";
 
 jest.mock("../utils/buildUrl", () => {
@@ -20,19 +21,71 @@ jest.mock("../utils/buildUrl", () => {
 	return modExport;
 });
 
+describe("getAppModules", () => {
+	it("creates a RSAA to fetch application modules", () => {
+		expect(getAppModules, "when called", "to exhaustively satisfy", {
+			[RSAA]: {
+				types: [GET_APPLICATION_MODULES_REQUEST, GET_APPLICATION_MODULES_SUCCESS, GET_APPLICATION_MODULES_FAILURE],
+				endpoint: 'URL: modules/byApplicationName/anApplication ""',
+				method: "GET",
+				body: undefined,
+				credentials: "include",
+				bailout: expect.it("to be a function"),
+				headers: {
+					Accept: "application/json; charset=utf-8",
+					"Content-Type": "application/json",
+				},
+				options: { redirect: "follow" },
+			},
+		});
+	});
+});
+
 describe("getScopes", () => {
-	beforeEach(() => {});
-	afterEach(() => {});
+	let originalApplication;
+
+	beforeEach(() => {
+		originalApplication = global.OVERTURE_APPLICATION;
+	});
+
+	afterEach(() => {
+		global.OVERTURE_APPLICATION = originalApplication;
+	});
 
 	it("creates a RSAA to fetch authorized scope tree", () =>
-		expect(getScopes, "when called", "to exhaustively satisfy", {
+		expect(getScopes, "when called with", ["aModule"], "to exhaustively satisfy", {
 			[RSAA]: {
-				types: [GET_SCOPES_REQUEST, GET_SCOPES_SUCCESS, GET_SCOPES_FAILURE],
+				types: [
+					{ type: "GET_SCOPES_REQUEST", meta: { module: "aModule" } },
+					{ type: "GET_SCOPES_SUCCESS", meta: { module: "aModule" } },
+					{ type: "GET_SCOPES_FAILURE", meta: { module: "aModule" } },
+				],
 				endpoint: "URL: my/scope/aModule/tree {}",
 				method: "GET",
 				body: undefined,
 				credentials: "include",
-				bailout: expect.it("to be a function"),
+				bailout: false,
+				headers: {
+					Accept: "application/json; charset=utf-8",
+					"Content-Type": "application/json",
+				},
+				options: { redirect: "follow" },
+			},
+		}));
+
+	it("creates a RSAA to fetch authorized scope tree for system module", () =>
+		expect(getScopes, "when called with", ["System"], "to exhaustively satisfy", {
+			[RSAA]: {
+				types: [
+					{ type: "GET_SCOPES_REQUEST", meta: { module: "System" } },
+					{ type: "GET_SCOPES_SUCCESS", meta: { module: "System" } },
+					{ type: "GET_SCOPES_FAILURE", meta: { module: "System" } },
+				],
+				endpoint: "URL: my/scope/Order/tree {}",
+				method: "GET",
+				body: undefined,
+				credentials: "include",
+				bailout: false,
 				headers: {
 					Accept: "application/json; charset=utf-8",
 					"Content-Type": "application/json",
@@ -42,14 +95,31 @@ describe("getScopes", () => {
 		}));
 
 	it("creates a RSAA to fetch default user scope", () =>
-		expect(getDefaultScope, "when called", "to exhaustively satisfy", {
+		expect(getDefaultScope, "when called with", ["aModule"], "to exhaustively satisfy", {
 			[RSAA]: {
 				types: [GET_MY_SCOPE_REQUEST, GET_MY_SCOPE_SUCCESS, GET_MY_SCOPE_FAILURE],
 				endpoint: 'URL: my/scope/aModule ""',
 				method: "GET",
 				body: undefined,
 				credentials: "include",
-				bailout: expect.it("to be a function"),
+				bailout: false,
+				headers: {
+					Accept: "application/json; charset=utf-8",
+					"Content-Type": "application/json",
+				},
+				options: { redirect: "follow" },
+			},
+		}));
+
+	it("creates a RSAA to fetch default user scope for System module", () =>
+		expect(getDefaultScope, "when called with", ["System"], "to exhaustively satisfy", {
+			[RSAA]: {
+				types: [GET_MY_SCOPE_REQUEST, GET_MY_SCOPE_SUCCESS, GET_MY_SCOPE_FAILURE],
+				endpoint: 'URL: my/scope/Order ""',
+				method: "GET",
+				body: undefined,
+				credentials: "include",
+				bailout: false,
 				headers: {
 					Accept: "application/json; charset=utf-8",
 					"Content-Type": "application/json",
@@ -59,11 +129,11 @@ describe("getScopes", () => {
 		}));
 
 	it("throws an error if no class found on DOM element", () => {
-		global.OVERTURE_MODULE = "";
+		global.OVERTURE_APPLICATION = "";
 		expect(
-			() => expect(validateOvertureModule, "when called with", []),
+			() => expect(validateOvertureApplication, "when called with", []),
 			"to throw",
-			'"overtureModule.name" is missing in the configuration.',
+			'"overtureApplication.name" is missing in the configuration.',
 		);
 	});
 });

package/src/components/AppFrame/About.js

@@ -8,7 +8,7 @@ import Text from "../Text";
 import withClickOutside from "../../hocs/withClickOutside";
 import useViewState from "../../hooks/useViewState";
 import bgImage from "../../content/aboutBackground.png";
-import logoImage from "../../content/aboutLogo.png";
+import logoImage from "../../content/orckestra-logo-white.png";
 import close from "../../content/close.png";
 import { getVersionSelector } from "../../selectors/versionInfo";
 import { currentLocaleOrDefault } from "../../selectors/locale";
@@ -101,7 +101,7 @@ export const About = ({ currentApplication }) => {
 			<CloseButton onClick={closeAboutBox}>
 				<img src={close} alt="X" />
 			</CloseButton>
-			<img src={logoImage} alt="Orckestra" />
+			<img src={logoImage} width="250" alt="Orckestra" />
 			<AboutParagraph>
 				<Text
 					message={{
@@ -164,7 +164,14 @@ export const About = ({ currentApplication }) => {
 				</AboutLink>
 			</AboutParagraph>
 			<AboutParagraph>
-				<Text message={sharedMessages.copyright} />
+				<Text
+					message={{
+						...sharedMessages.copyright,
+						values: {
+							year: new Date().getFullYear(),
+						},
+					}}
+				/>
 				<br />
 				<Text message={sharedMessages.allRightsReserved} />
 			</AboutParagraph>

package/src/components/AppFrame/AppFrame.test.js

@@ -202,6 +202,15 @@ describe("AppFrame", () => {
 					bar: true,
 				},
 			},
+			modules: {
+				tree: "modulesTree",
+				visibleModules: ["a", "module123", "test1", "test2", "test3"],
+				lastScopeAndModuleSelection: {
+					scope: "Norway",
+					moduleName: "Profiles",
+					routingPerformed: false,
+				},
+			},
 			settings: { defaultScope: "myScope", defaultApp: "12" },
 			versionInfo: { version: "4.2", defaultHelpUrl: "help_url", moduleHelpUrls: [] },
 			view: { scopeSelector: { filter: "1" }, __prefsDialog: { show: false } },

package/src/components/AppFrame/MenuItem.js

@@ -1,5 +1,4 @@
 import React, { useRef } from "react";
-import { useSelector } from "react-redux";
 import styled, { css } from "styled-components";
 import transition from "styled-transition-group";
 import { Link } from "react-router-dom";
@@ -124,13 +123,11 @@ const MenuItem = ({
 	alert,
 	pageScopeSelector,
 	closingTabHandler,
-	hide,
+	hide, // This props is not used in the component, but we need to destructure it so props can be used in ItemWrapper
+	isHidden = false,
 	href,
 	...props
 }) => {
-	let hideSelector = state => (typeof hide === "function" ? hide(state) : hide ?? false);
-	const isHidden = useSelector(hideSelector);
-
 	let ItemWrapper = Block;
 	if (props.menuToggle) {
 		ItemWrapper = BlockWithA;
@@ -139,6 +136,7 @@ const MenuItem = ({
 	if (alert && alert.message) {
 		alertMessage.current = alert.message;
 	}
+
 	return (
 		!isHidden && (
 			<ItemWrapper to={href} {...props}>

package/src/components/AppFrame/MenuItem.test.js

@@ -168,7 +168,7 @@ describe("MenuItem", () => {
 			</MemoryRouter>,
 		));
 
-	it("shows nothing if hide is true", () => {
+	it("shows nothing if isHidden is true", () => {
 		expect(
 			<Provider store={store}>
 				<MemoryRouter>
@@ -178,29 +178,7 @@ describe("MenuItem", () => {
 						icon="cake"
 						label="Test"
 						alert={{ message: "Test message", type: "warn" }}
-						hide={true}
-					/>
-				</MemoryRouter>
-			</Provider>,
-			"when mounted",
-			"to equal",
-			null,
-		);
-	});
-
-	it("shows nothing if hide selector returns true", () => {
-		const hide = state => true;
-
-		expect(
-			<Provider store={store}>
-				<MemoryRouter>
-					<MenuItem
-						id="test"
-						href="/foo/test"
-						icon="cake"
-						label="Test"
-						alert={{ message: "Test message", type: "warn" }}
-						hide={hide}
+						isHidden={true}
 					/>
 				</MemoryRouter>
 			</Provider>,

package/src/components/AppFrame/Sidebar.js

@@ -5,6 +5,7 @@ import { useLocation } from "react-router-dom";
 import { getThemeProp } from "../../utils";
 import { selectPrependHrefConfig } from "../../selectors/navigation";
 import MenuItem from "./MenuItem";
+import { getScopeModuleInformationSelector } from "../../selectors/modules";
 
 export const Bar = styled.div`
 	box-sizing: border-box;
@@ -40,7 +41,13 @@ const useEnhancement = id => {
 	};
 };
 
-export const EnhancedMenuItem = ({ id, ...props }) => <MenuItem {...props} {...useEnhancement(id)} />;
+export const EnhancedMenuItem = ({ id, ...props }) => {
+	const defaultModule = useSelector(getScopeModuleInformationSelector);
+
+	const isHidden = !defaultModule.visibleModules.includes(id);
+
+	return <MenuItem {...props} {...useEnhancement(id)} isHidden={isHidden} />;
+};
 
 const LogoSvg = styled.svg`
 	flex: 0 0 auto;
@@ -53,17 +60,9 @@ const LogoSvg = styled.svg`
 `;
 
 export const Logo = () => (
-	<LogoSvg>
-		<path
-			id="orckestra-logo"
-			d="M10.3,3.8c-1.7,0-3.5,0.7-4.7,1.9L2.8,3c1.9-1.9,4.6-3,7.5-3c0.6,0,1.3,0.1,1.9,0.2
-		l-0.9,3.7C11,3.8,10.6,3.8,10.3,3.8L10.3,3.8z M16.1,7.5c-1.4-1.4-3.2-2.1-5.1-2.1c-2,0-3.9,0.8-5.3,2.3c-1.2,1.2-1.9,2.8-2,4.5H0
-		c0.1-2.7,1-4.9,3-7c0.2-0.2,0.5-0.5,0.7-0.7l1.8,1.8l0.1-0.1c1.2-1.2,2.9-1.9,4.6-1.9c0.4,0,0.8,0,1.2,0.1l0.2,0l0.6-2.5
-		c2.3,0.3,4.3,1.2,6,2.8l0.3,0.3c2,1.9,3.1,4.5,3.3,7.3h-3.6C18.1,10.4,17.3,8.7,16.1,7.5L16.1,7.5z M19.8,3.3
-		c2.9,2.8,4.4,6.8,4,10.8c-0.3,2.8-1.5,5.4-3.4,7.6L17,19.1c0.4-0.4,0.7-0.8,1-1.2c0.9-1.2,1.4-2.7,1.6-4.2c0-0.4,0.1-0.7,0-1.1h2.7
-		l0-0.2c-0.1-3-1.4-5.7-3.5-7.8l-0.2-0.2L19.8,3.3L19.8,3.3z M16.5,18.9l-0.2,0.2l2.6,2c-2.3,2.6-5.6,4-9.1,3.8l0-4
-		c0.2,0,0.4,0,0.6,0h0c2.6,0,5-1.4,6.4-3.6l0.7,0.5C17.2,18.2,16.9,18.6,16.5,18.9L16.5,18.9z"
-		/>
+	<LogoSvg viewBox="0 0 260 260">
+		<path d="M1.11,125.62C1.11,74.94,40.22,39,93.72,39S186,74.94,186,125.62s-38.79,86.66-92.29,86.66S1.11,176.3,1.11,125.62Zm135.47,0c0-29.1-18.46-46.62-42.86-46.62S50.54,96.52,50.54,125.62s18.77,46.62,43.18,46.62S136.58,154.72,136.58,125.62Z" />
+		<circle cx="227.6" cy="181.13" r="31.29" />
 	</LogoSvg>
 );