@beinformed/ui 1.13.2 → 1.13.6

package/lib/utils/helpers/objects.js.map

@@ -1 +1 @@
-{"version":3,"sources":["../../../src/utils/helpers/objects.js"],"names":["isPlainObject","value","Object","prototype","toString","call","has","obj","propName","get","path","defaultValue","String","split","reduce","acc","v"],"mappings":";;;;;;;;;;;AACA;;AAEA;AACA;AACO,IAAMA,aAAa,GAAG,SAAhBA,aAAgB,CAACC,KAAD;AAAA,SAC3BC,MAAM,CAACC,SAAP,CAAiBC,QAAjB,CAA0BC,IAA1B,CAA+BJ,KAA/B,MAA0C,iBADf;AAAA,CAAtB;AAGP;AACA;;;;;AACO,IAAMK,GAAG,GAAG,SAANA,GAAM,CAACC,GAAD,EAAWC,QAAX,EAAyC;AAC1D,MAAI,CAACR,aAAa,CAACO,GAAD,CAAlB,EAAyB;AACvB,WAAO,KAAP;AACD;;AAED,SAAO,mBAAKA,GAAL,EAAUC,QAAV,CAAP;AACD,CANM;AAQP;AACA;AACA;AACA;;;;;AACO,IAAMC,GAAG,GAAG,SAANA,GAAM,CAACF,GAAD,EAAeG,IAAf,EAA6BC,YAA7B;AAAA,SACjBC,MAAM,CAACF,IAAD,CAAN,CACGG,KADH,CACS,GADT,EAEGC,MAFH,CAEU,UAACC,GAAD,EAAMC,CAAN,EAAY;AAClB,QAAI,sBAAOD,GAAP,MAAe,QAAnB,EAA6B;AAAA;;AAC3B,uBAAOA,GAAG,CAACC,CAAD,CAAV,2CAAiBL,YAAjB;AACD;;AACD,WAAOA,YAAP;AACD,GAPH,EAOKJ,GAPL,CADiB;AAAA,CAAZ","sourcesContent":["// @flow\nimport _has from \"lodash/has\";\n\n/**\n */\nexport const isPlainObject = (value: any): boolean =>\n  Object.prototype.toString.call(value) === \"[object Object]\";\n\n/**\n */\nexport const has = (obj: any, propName: string): boolean => {\n  if (!isPlainObject(obj)) {\n    return false;\n  }\n\n  return _has(obj, propName);\n};\n\n/**\n * Retrieve property of an object\n * When path is only one deep better use optional chaining\n */\nexport const get = (obj: { ... }, path: string, defaultValue?: any): any =>\n  String(path)\n    .split(\".\")\n    .reduce((acc, v) => {\n      if (typeof acc === \"object\") {\n        return acc[v] ?? defaultValue;\n      }\n      return defaultValue;\n    }, obj);\n"],"file":"objects.js"}
+{"version":3,"sources":["../../../src/utils/helpers/objects.js"],"names":["isPlainObject","value","has","obj","propName","get","path","defaultValue","String","split","reduce","acc","v"],"mappings":";;;;;;;;;;;AACA;;AACA;;AAEA;AACA;AACO,IAAMA,aAAa,GAAG,SAAhBA,aAAgB,CAACC,KAAD;AAAA,SAAyB,6BAAeA,KAAf,CAAzB;AAAA,CAAtB;AAEP;AACA;;;;;AACO,IAAMC,GAAG,GAAG,SAANA,GAAM,CAACC,GAAD,EAAWC,QAAX,EAAyC;AAC1D,MAAI,CAACJ,aAAa,CAACG,GAAD,CAAlB,EAAyB;AACvB,WAAO,KAAP;AACD;;AAED,SAAO,mBAAKA,GAAL,EAAUC,QAAV,CAAP;AACD,CANM;AAQP;AACA;AACA;AACA;;;;;AACO,IAAMC,GAAG,GAAG,SAANA,GAAM,CAACF,GAAD,EAAeG,IAAf,EAA6BC,YAA7B;AAAA,SACjBC,MAAM,CAACF,IAAD,CAAN,CACGG,KADH,CACS,GADT,EAEGC,MAFH,CAEU,UAACC,GAAD,EAAMC,CAAN,EAAY;AAClB,QAAI,sBAAOD,GAAP,MAAe,QAAnB,EAA6B;AAAA;;AAC3B,uBAAOA,GAAG,CAACC,CAAD,CAAV,2CAAiBL,YAAjB;AACD;;AACD,WAAOA,YAAP;AACD,GAPH,EAOKJ,GAPL,CADiB;AAAA,CAAZ","sourcesContent":["// @flow\nimport _has from \"lodash/has\";\nimport _isPlainObject from \"lodash/isPlainObject\";\n\n/**\n */\nexport const isPlainObject = (value: any): boolean => _isPlainObject(value);\n\n/**\n */\nexport const has = (obj: any, propName: string): boolean => {\n  if (!isPlainObject(obj)) {\n    return false;\n  }\n\n  return _has(obj, propName);\n};\n\n/**\n * Retrieve property of an object\n * When path is only one deep better use optional chaining\n */\nexport const get = (obj: { ... }, path: string, defaultValue?: any): any =>\n  String(path)\n    .split(\".\")\n    .reduce((acc, v) => {\n      if (typeof acc === \"object\") {\n        return acc[v] ?? defaultValue;\n      }\n      return defaultValue;\n    }, obj);\n"],"file":"objects.js"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@beinformed/ui",
-  "version": "1.13.2",
+  "version": "1.13.6",
   "description": "Toolbox for be informed javascript layouts",
   "license": "SEE LICENSE IN LICENSE.md",
   "bugs": "http://support.beinformed.com",
@@ -78,7 +78,7 @@
     "styled-components": "^5.2.0"
   },
   "dependencies": {
-    "@babel/runtime-corejs3": "^7.16.8",
+    "@babel/runtime-corejs3": "^7.17.2",
     "big.js": "^6.1.1",
     "date-fns": "^2.28.0",
     "deepmerge": "^4.2.2",
@@ -95,44 +95,44 @@
     "setimmediate": "^1.0.5"
   },
   "devDependencies": {
-    "@babel/cli": "^7.16.8",
-    "@babel/core": "^7.16.12",
-    "@babel/eslint-parser": "^7.16.5",
+    "@babel/cli": "^7.17.0",
+    "@babel/core": "^7.17.2",
+    "@babel/eslint-parser": "^7.17.0",
     "@babel/eslint-plugin": "^7.16.5",
     "@babel/plugin-syntax-dynamic-import": "^7.8.3",
-    "@babel/plugin-transform-runtime": "^7.16.10",
+    "@babel/plugin-transform-runtime": "^7.17.0",
     "@babel/preset-env": "^7.16.11",
     "@babel/preset-flow": "^7.16.7",
     "@babel/preset-react": "^7.16.7",
     "@commitlint/cli": "^16.1.0",
     "@commitlint/config-conventional": "^16.0.0",
     "@testing-library/react-hooks": "^7.0.2",
-    "auditjs": "^4.0.35",
-    "babel-jest": "^27.4.6",
+    "auditjs": "^4.0.36",
+    "babel-jest": "^27.5.1",
     "babel-plugin-styled-components": "^2.0.2",
     "cherry-pick": "^0.5.0",
     "cross-env": "^7.0.3",
     "documentation": "^13.2.5",
-    "eslint": "^8.7.0",
+    "eslint": "^8.8.0",
     "eslint-config-prettier": "^8.3.0",
     "eslint-plugin-babel": "^5.3.1",
-    "eslint-plugin-flowtype": "^8.0.3",
+    "eslint-plugin-ft-flow": "^2.0.1",
     "eslint-plugin-import": "^2.25.4",
     "eslint-plugin-jest": "^26.0.0",
-    "eslint-plugin-jsdoc": "^37.7.0",
+    "eslint-plugin-jsdoc": "^37.7.1",
     "eslint-plugin-react": "^7.28.0",
     "eslint-plugin-react-hooks": "^4.3.0",
-    "flow-bin": "0.152.0",
+    "flow-bin": "^0.171.0",
     "flow-copy-source": "^2.0.9",
     "flow-typed": "^3.6.1",
     "glob": "^7.2.0",
     "history": "^4.0.0",
     "husky": "^7.0.4",
-    "jest": "^27.4.7",
+    "jest": "^27.5.1",
     "jest-junit": "^13.0.0",
     "jest-sonar-reporter": "^2.0.0",
-    "jscodeshift": "^0.13.0",
-    "lint-staged": "^12.3.2",
+    "jscodeshift": "^0.13.1",
+    "lint-staged": "^12.3.3",
     "polished": "^4.1.4",
     "prettier": "^2.5.1",
     "react": "^17.0.2",

package/src/constants/LayoutHintConfig.js

@@ -109,6 +109,15 @@ export const LayoutHintConfiguration = {
     link: "",
     component: ["attribute/number"],
   },
+  IGNORE_FORMAT_SETTING: {
+    hint: "ignore-format-setting",
+    description: {
+      NL: "Maakt het mogelijk om de global date format setting te negeren voor een specifiek attribuut",
+      EN: "Makes it possible to ignore the global date format for a specific attribute",
+    },
+    link: "",
+    component: ["attribute/date", "attribute/datetime"],
+  },
   CREATE_ACTION: {
     hint: "create",
     description: {

package/src/constants/LayoutHints.js

@@ -53,6 +53,10 @@ export const SORT_OPTIONS: string = getHint("SORT_OPTIONS");
  * Layout-hint: Marks an attribute as a title attribute
  */
 export const TITLE: string = getHint("TITLE");
+/**
+ * Layout-hint: Ignore the global input format for specific attribute
+ */
+export const IGNORE_FORMAT_SETTING: string = getHint("IGNORE_FORMAT_SETTING");
 
 /*
  * --------------------------------

package/src/hooks/__tests__/useAuthentication.spec.js

@@ -72,6 +72,7 @@ describe("authentication hooks", () => {
           href: new Href("/", "Application"),
         }),
       }),
+      { type: "FINISH_PROGRESS" },
       { type: "AUTHENTICATION_SUCCESS" },
       { type: "FINISH_PROGRESS" },
     ]);

package/src/models/attributes/AttributeContent.js

@@ -62,7 +62,7 @@ class AttributeContent {
     if (this._content?.elements) {
       return this._content.elements?.map((element) => {
         if ("propertyElement" in element) {
-          // $FlowIssue
+          // $FlowIssue[prop-missing]
           const { label, layouthint, properties } = element.propertyElement;
 
           return {
@@ -76,7 +76,7 @@ class AttributeContent {
 
         if ("textFragmentElement" in element) {
           const { label, layouthint, textfragments } =
-            // $FlowIssue
+            // $FlowIssue[prop-missing]
             element.textFragmentElement;
           return {
             textFragmentElement: {
@@ -91,7 +91,7 @@ class AttributeContent {
         }
 
         if ("contentElement" in element) {
-          // $FlowIssue
+          // $FlowIssue[prop-missing]
           const { label, layouthint, sections } = element.contentElement;
           return {
             contentElement: {

package/src/models/attributes/AttributeModel.js

@@ -26,7 +26,12 @@ import {
 } from "../../exceptions";
 
 import type { MessageParameters } from "../../i18n/types";
-import type { AttributeType, FormErrorAnchor, ModularUIModel } from "../types";
+import type {
+  AttributeType,
+  FormErrorAnchor,
+  IModelWithChildModels,
+  ModularUIModel,
+} from "../types";
 import type LinkModel from "../links/LinkModel";
 import type ContentConfiguration from "../contentconfiguration/ContentConfiguration";
 import type ContentConfigurationElements from "../contentconfiguration/ContentConfigurationElements";
@@ -37,7 +42,10 @@ import AttributeContent from "./AttributeContent";
 /**
  * Attribute model, base model for all kind of attributes
  */
-export default class AttributeModel extends BaseModel {
+export default class AttributeModel
+  extends BaseModel
+  implements IModelWithChildModels
+{
   _label: string;
   _lastModification: number = 0;
 

package/src/models/attributes/BooleanAttributeModel.js

@@ -142,7 +142,10 @@ export default class BooleanAttributeModel extends AttributeModel {
    * Getting all enabled options
    */
   get selected(): Array<string> {
-    return this.options.selected.map<string>((option) => option.code);
+    // $FlowFixMe[incompatible-call]
+    return this.options.selected.map<string, ChoiceAttributeOptionModel>(
+      (option: ChoiceAttributeOptionModel) => option.code
+    );
   }
 
   /**

package/src/models/attributes/ChoiceAttributeModel.js

@@ -149,7 +149,10 @@ export default class ChoiceAttributeModel extends AttributeModel {
    * Getting all enabled options
    */
   get selected(): Array<string> {
-    return this.options.selected.map<string>((option) => option.code);
+    // $FlowFixMe[incompatible-call]
+    return this.options.selected.map<string, ChoiceAttributeOptionModel>(
+      (option: ChoiceAttributeOptionModel) => option.code
+    );
   }
 
   /**

package/src/models/attributes/ChoiceAttributeOptionCollection.js

@@ -220,7 +220,7 @@ class ChoiceAttributeOptionCollection extends ResourceCollection<ChoiceAttribute
   get selected(): Array<ChoiceAttributeOptionModel> {
     const selectedOptions = [];
 
-    this.collection.forEach((option) => {
+    this.collection.forEach((option: ChoiceAttributeOptionModel) => {
       if (option.selected && option.selected === true) {
         selectedOptions.push(option);
       }

package/src/models/attributes/ChoiceAttributeOptionModel.js

@@ -13,13 +13,16 @@ import AttributeCollection from "./AttributeCollection";
 import ConceptDetailModel from "../concepts/ConceptDetailModel";
 import AttributeContent from "./AttributeContent";
 
-import type { ModularUIModel } from "../types";
+import type { IModelWithChildModels, ModularUIModel } from "../types";
 import type ListItemModel from "../list/ListItemModel";
 import type LinkModel from "../links/LinkModel";
 
 /**
  */
-class ChoiceAttributeOptionModel extends BaseModel {
+class ChoiceAttributeOptionModel
+  extends BaseModel
+  implements IModelWithChildModels
+{
   _code: string;
   _isSelected: boolean;
   _referenceDate: ISO_DATE;

package/src/models/attributes/CompositeAttributeChildCollection.js

@@ -165,6 +165,7 @@ class CompositeAttributeChildCollection extends ResourceCollection<AttributeType
    */
   get initvalue(): any {
     if (this.first instanceof CompositeAttributeChildCollection) {
+      // $FlowFixMe[missing-type-arg]
       return this.collection.map<Array<Object>>(
         (collection) => collection.initvalue
       );
@@ -195,7 +196,6 @@ class CompositeAttributeChildCollection extends ResourceCollection<AttributeType
 
     this.collection.forEach((attribute) => {
       if (attribute.formdata !== null && !attribute.readonly) {
-        // $FlowIgnore[cannot-spread-indexer]
         children = {
           ...children,
           ...attribute.formdata,

package/src/models/attributes/CompositeAttributeModel.js

@@ -375,6 +375,7 @@ class CompositeAttributeModel extends AttributeModel {
       (this.key === otherAttribute.key &&
         this.parentKey === otherAttribute.parentKey) ||
       this.children.some(
+        //$FlowFixMe[method-unbinding]
         (child) => child.equals && child.equals(otherAttribute)
       )
     );

package/src/models/attributes/DatetimeAttributeModel.js

@@ -18,6 +18,7 @@ import DateBoundaryConstraint from "./input-constraints/DateBoundaryConstraint";
 
 import { ATTRIBUTE_WIDTH, ISO_DATE_FORMAT } from "../../constants";
 import { getSetting } from "../../constants/Settings";
+import { IGNORE_FORMAT_SETTING } from "../../constants/LayoutHints";
 
 import type { FormErrorAnchor, IConstraintModel } from "../types";
 
@@ -239,10 +240,12 @@ class DatetimeAttributeModel extends StringAttributeModel {
     }
 
     // setting for date input
-    const dateInputFormatSetting = getSetting("DATE_INPUT_FORMAT", "");
-
-    if (dateInputFormatSetting) {
-      return dateInputFormatSetting;
+    const ignoreFormatSetting = this.layouthint.has(IGNORE_FORMAT_SETTING);
+    if (!ignoreFormatSetting) {
+      const dateInputFormatSetting = getSetting("DATE_INPUT_FORMAT", "");
+      if (dateInputFormatSetting) {
+        return dateInputFormatSetting;
+      }
     }
 
     let format = this.format;

package/src/models/base/BaseCollection.js

@@ -154,9 +154,9 @@ class BaseCollection<T> {
    */
   map<U>(
     callbackfn: (value: T, index: number, array: Array<T>) => U,
-    thisArg?: any
+    thisArg?: T
   ): Array<U> {
-    return this.collection.map<U>(callbackfn, thisArg);
+    return this.collection.map<U, ?T>(callbackfn, thisArg);
   }
 
   /**

package/src/models/base/ResourceCollection.js

@@ -16,11 +16,8 @@ class ResourceCollection<T: IModelWithChildModels> extends BaseCollection<T> {
     const initialChildModelLinks = [];
 
     this.collection.forEach((item: T) => {
-      if (
-        item &&
-        item.getInitialChildModelLinks &&
-        typeof item.getInitialChildModelLinks === "function"
-      ) {
+      // $FlowFixMe[method-unbinding]
+      if (typeof item?.getInitialChildModelLinks === "function") {
         initialChildModelLinks.push(...item.getInitialChildModelLinks());
       }
     });
@@ -33,7 +30,8 @@ class ResourceCollection<T: IModelWithChildModels> extends BaseCollection<T> {
    */
   setChildModels(models: Array<ModularUIModel>) {
     this.collection.forEach((item) => {
-      if (typeof item.setChildModels === "function") {
+      // $FlowFixMe[prop-missing]
+      if (typeof item["setChildModels"] === "function") {
         item.setChildModels(models);
       }
     });

package/src/models/base/ResourceModel.js

@@ -200,8 +200,9 @@ class ResourceModel extends BaseModel implements IModelWithChildModels {
       ...super.dehydrate(),
       key: this._key,
       locale: this._locale,
-      childModels: this._childModels.map<ModularUIModel>((childModel) =>
-        childModel.dehydrate()
+      // $FlowFixMe[missing-type-arg]
+      childModels: this._childModels.map<ModularUIModel>(
+        (childModel: ModularUIModel) => childModel.dehydrate()
       ),
     };
   }

package/src/models/concepts/BusinessScenarioModel.js

@@ -1,5 +1,6 @@
 // @flow
 import ConceptDetailModel from "./ConceptDetailModel";
+import ConceptTypeDetailModel from "./ConceptTypeDetailModel";
 
 import type { ModularUIResponse } from "../../modularui";
 import type { ModularUIModel } from "../types";
@@ -58,7 +59,7 @@ class BusinessScenarioModel extends ConceptDetailModel {
   setChildModels(models: Array<ModularUIModel>) {
     // $FlowExpectedError[incompatible-type]
     this.conceptType = models.find(
-      (model) => model.type === "ConceptTypeDetail"
+      (model) => model instanceof ConceptTypeDetailModel
     );
 
     const scenarioSteps = [];

package/src/models/concepts/ConceptDetailModel.js

@@ -2,6 +2,7 @@
 import ResourceModel from "../base/ResourceModel";
 import ConceptRelationCollection from "./ConceptRelationCollection";
 import SourceReferenceCollection from "./SourceReferenceCollection";
+import ConceptTypeDetailModel from "./ConceptTypeDetailModel";
 
 import { TIMEVERSION_FILTER_NAME } from "../../constants/Constants";
 
@@ -14,7 +15,6 @@ import type {
 import type { ModularUIResponse } from "../../modularui";
 import type Href from "../href/Href";
 import type LinkModel from "../links/LinkModel";
-import type ConceptTypeDetailModel from "./ConceptTypeDetailModel";
 
 /**
  * Model for concept details, available through modelcatalog
@@ -70,7 +70,7 @@ export default class ConceptDetailModel extends ResourceModel {
   setChildModels(models: Array<ModularUIModel>) {
     // $FlowExpectedError[incompatible-type]
     const conceptTypeModel: ?ConceptTypeDetailModel = models.find(
-      (model) => model.type === "ConceptTypeDetail"
+      (model) => model instanceof ConceptTypeDetailModel
     );
 
     if (conceptTypeModel) {

package/src/models/concepts/ConceptLinkModel.js

@@ -5,12 +5,15 @@ import LinkCollection from "../links/LinkCollection";
 import ConceptTypeDetailModel from "./ConceptTypeDetailModel";
 import { TIMEVERSION_FILTER_NAME } from "../../constants/Constants";
 
-import type { ModularUIModel } from "../types";
+import type { IModelWithChildModels, ModularUIModel } from "../types";
 import type Href from "../href/Href";
 /**
  * Link to a concept
  */
-export default class ConceptLinkModel extends BaseModel {
+export default class ConceptLinkModel
+  extends BaseModel
+  implements IModelWithChildModels
+{
   _links: ?LinkCollection;
   _entryDate: ?ISO_DATE;
   _conceptType: ?ConceptTypeDetailModel;

package/src/models/concepts/ConceptRelationModel.js

@@ -2,12 +2,16 @@
 import ConceptLinkModel from "./ConceptLinkModel";
 import LinkModel from "../links/LinkModel";
 
-import type { ModularUIModel, textfragmentJSON } from "../types";
+import type {
+  IModelWithChildModels,
+  ModularUIModel,
+  textfragmentJSON,
+} from "../types";
 
 /**
  * Concept relation model
  */
-export default class ConceptRelationModel {
+export default class ConceptRelationModel implements IModelWithChildModels {
   _relation: Object;
   _concept: ConceptLinkModel;
 

package/src/models/content/ContentLinkModel.js

@@ -33,6 +33,7 @@ export default class ContentLinkModel
 
   /**
    */
+  // $FlowFixMe[method-unbinding]
   getInitialChildModelLinks(): Array<LinkModel> {
     if (this.contentTypeLink) {
       this.contentTypeLink.isCacheable = true;
@@ -44,6 +45,7 @@ export default class ContentLinkModel
 
   /**
    */
+  // $FlowFixMe[method-unbinding]
   setChildModels(models: Array<ModularUIModel>) {
     if (this.contentTypeLink) {
       const href = this.contentTypeLink.href;

package/src/models/href/Href.js

@@ -13,6 +13,7 @@ type HrefObject = {
   _resourcetype: string,
   _method: $Keys<typeof HTTP_METHODS>,
   _state: ?{ [key: string]: any },
+  pathname: ?string,
 };
 
 export type HrefInput = Href | HrefObject | LocationShape | string;
@@ -61,7 +62,7 @@ class Href {
   /**
    */
   setFromLocationOrObject(href: LocationShape | HrefObject) {
-    if (href.pathname) {
+    if (typeof href["pathname"] === "string") {
       this.path = href.pathname;
     } else if (typeof href._path === "string") {
       this.path = href._path;

package/src/models/types.js

@@ -149,8 +149,8 @@ export interface ILayoutHintRule {
 }
 
 export interface IModelWithChildModels {
-  +getInitialChildModelLinks: () => Array<LinkModel>;
-  +setChildModels: (models: Array<ModularUIModel>) => void;
+  getInitialChildModelLinks(): Array<LinkModel>;
+  setChildModels(models: Array<ModularUIModel>): void;
 }
 
 export type FileEntryType = {

package/src/react-server/__tests__/serverUtil.spec.js

@@ -16,6 +16,20 @@ global.preferencesProvider = {
 
 const mockStore = configureMockStore();
 
+const getExpectedLocales = () =>
+  new Locales([
+    ...availableLocales.map((locale) =>
+      locale.code === "nl"
+        ? {
+            ...locale,
+            messages: {
+              message_nl: "Dutch message",
+            },
+          }
+        : locale
+    ),
+  ]);
+
 describe("serverUtil", () => {
   it("handles setI18n", () => {
     const store = mockStore({ i18n: {} });
@@ -39,25 +53,12 @@ describe("serverUtil", () => {
       }
     );
 
-    const expectedLocales = new Locales([
-      ...availableLocales.map((locale) =>
-        locale.code === "nl"
-          ? {
-              ...locale,
-              messages: {
-                message_nl: "Dutch message",
-              },
-            }
-          : locale
-      ),
-    ]);
-
     expect(store.getActions()).toStrictEqual([
       {
         type: "SET_LOCALES",
         payload: {
           locale: "en",
-          locales: expectedLocales,
+          locales: getExpectedLocales(),
         },
       },
     ]);

package/src/react-server/renderSSRComplete.js

@@ -21,6 +21,8 @@ import type { Theme } from "../react-theme/types";
 import type { ReduxStore } from "../redux/types";
 import type Href from "../models/href/Href";
 import type { TemplateProps } from "./htmlpage";
+import type { Location } from "react-router";
+
 type Props = {
   store: ReduxStore,
   theme: Theme,
@@ -46,7 +48,8 @@ const renderSSRComplete = ({
 
   const sheet = new ServerStyleSheet();
 
-  // $FlowExpectedError[incompatible-type]
+  // $FlowFixMe[incompatible-type]
+  // $FlowFixMe[prop-missing]
   const location: Location = requestHref.toLocation();
 
   return createSSRComplete(store, () =>

package/src/react-server/serverNoSSR.js

@@ -56,7 +56,7 @@ const serverNoSSR = ({
 
   if (theme) {
     const storedTheme = store.getState()?.preferences?.theme;
-    // $FlowFixMe
+    // $FlowFixMe[incompatible-call]
     createTheme(storedTheme, theme);
   }
 

package/src/redux/_modularui/__tests__/actions.spec.js

@@ -152,6 +152,7 @@ describe("modularui actions", () => {
           targetModel: undefined,
         }),
       },
+      { type: "FINISH_PROGRESS" },
     ]);
   });
 });

package/src/redux/_modularui/actions.js

@@ -73,7 +73,8 @@ const loadModelSuccessAction = (
   modelToUpdate
 ): UpdateModelAction | SetModelAction => {
   if (modelToUpdate) {
-    if (typeof modelToUpdate.update === "function") {
+    // $FlowFixMe[prop-missing]
+    if (typeof modelToUpdate["update"] === "function") {
       const clonedModel = modelToUpdate.clone();
       clonedModel.update(model);
 
@@ -140,17 +141,16 @@ export const loadModularUI =
     dispatch(startProgress());
 
     const loadModelPromise = dispatch(loadModel(key, href, options));
-    if (typeof loadModelPromise !== "function" && loadModelPromise.then) {
-      return loadModelPromise
-        .then((response) => {
-          if (response?.type === "FINISH_PROGRESS") {
-            dispatch(updateStatus(key, MODULARUI_STATUS.FINISHED));
-          }
-
-          return dispatch(finishProgress());
-        })
-        .catch((error) => dispatch(handleError(error)));
-    }
+
+    return Promise.resolve(loadModelPromise)
+      .then((response) => {
+        if (response?.type === "FINISH_PROGRESS") {
+          dispatch(updateStatus(key, MODULARUI_STATUS.FINISHED));
+        }
+
+        return dispatch(finishProgress());
+      })
+      .catch((error) => dispatch(handleError(error)));
   };
 
 /**

package/src/redux/_modularui/reducer.js

@@ -74,7 +74,6 @@ const removeKey = (modelKey: string, state: ModularUIState): ModularUIState => {
   // eslint-disable-next-line no-unused-vars
   const { [modelKey]: _, ...newState } = state;
 
-  // $FlowExpectedError[incompatible-return]
   return newState;
 };
 

package/src/redux/actions/SignIn.js

@@ -66,17 +66,11 @@ export const login =
       .catch((error) => {
         if (error.id === "Error.ChangePasswordRequired") {
           const dispatchedReloadApplication = dispatch(reloadApplication());
-          if (
-            typeof dispatchedReloadApplication !== "function" &&
-            dispatchedReloadApplication.then
-          ) {
-            dispatchedReloadApplication
-              .then(() => {
-                Cache.addItem("auth", true);
-                return dispatch(changePassword());
-              })
-              .then(() => dispatch(finishProgress()));
-          }
+
+          return Promise.resolve(dispatchedReloadApplication).then(() => {
+            Cache.addItem("auth", true);
+            return dispatch(changePassword());
+          });
         }
 
         dispatch(loginFailed(error.id));

package/src/redux/actions/__tests__/Application.spec.js

@@ -27,6 +27,7 @@ describe("application actions", () => {
           href: new Href("/", "Application"),
         }),
       }),
+      { type: "FINISH_PROGRESS" },
     ]);
   });
 

package/src/redux/actions/__tests__/CustomErrorResponseHandler.spec.js

@@ -27,6 +27,7 @@ describe("custom error response handling", () => {
     const store = mockStore({});
 
     setCustomErrorResponseHandler((error) => {
+      // eslint-disable-next-line jest/no-conditional-in-test
       if (error.isRemoteServiceException) {
         return {
           type: "REMOTE_SERVICE_EXCEPTION",
@@ -50,6 +51,7 @@ describe("custom error response handling", () => {
     const store = mockStore({});
 
     setCustomErrorResponseHandler((error, defaultHandler) => (dispatch) => {
+      // eslint-disable-next-line jest/no-conditional-in-test
       if (error.isRemoteServiceException) {
         dispatch({
           type: "REMOTE_SERVICE_EXCEPTION",