@applicaster/quick-brick-core 12.0.0-alpha.5040434238 → 12.0.0-alpha.5050333009

package/App/NavigationProvider/NavigationProvider.tsx

@@ -64,6 +64,9 @@ import { ZappPipesEntryContext } from "@applicaster/zapp-react-native-ui-compone
 import { PlayNextState } from "@applicaster/zapp-react-native-utils/appUtils/playerManager/OverlayObserver/OverlaysObserver";
 import { inflateUrl } from "@applicaster/zapp-react-native-ui-components/Components/LinkHandler/LinkHandler";
 import { useModalStore } from "@applicaster/zapp-react-native-utils/modalState";
+import { useSubscriberFor } from "@applicaster/zapp-react-native-utils/reactHooks/useSubscriberFor";
+import { QBUIComponentEvents } from "@applicaster/zapp-react-native-ui-components/events";
+import { Hook } from "@applicaster/zapp-react-native-utils/appUtils/HooksManager/Hook";
 
 const logger = coreAppLogger.addSubsystem("Navigator");
 
@@ -73,17 +76,6 @@ type Props = {
   plugins?: QuickBrickPlugin[];
 };
 
-type EntryWithParent = ZappEntry & { parent?: EntryWithParent };
-
-type SingleEntry = {
-  data: EntryWithParent;
-  route: string;
-  id: string;
-  parentId?: string;
-};
-
-type EntryContextType = { [K in string]: SingleEntry };
-
 const navigationActions = {
   [ACTIONS.PUSH]: push,
   [ACTIONS.REPLACE]: replace,
@@ -120,9 +112,7 @@ export function NavigationProvider({ children }: Props) {
   const pathname = state?.location?.pathname;
   const pathnameRef = React.useRef(pathname);
 
-  const { context }: EntryContextType = React.useContext(
-    ZappPipesEntryContext.Context
-  );
+  const { context } = React.useContext(ZappPipesEntryContext.Context);
 
   const { openModal, dismissModal } = useModalStore();
 
@@ -130,6 +120,10 @@ export function NavigationProvider({ children }: Props) {
 
   const online = useConnectionInfo(true);
 
+  const replaceEvent = useSubscriberFor(
+    QBUIComponentEvents.navigatorReplaceItem
+  );
+
   React.useEffect(() => {
     stateRef.current = state;
     pathnameRef.current = pathname;
@@ -259,6 +253,7 @@ export function NavigationProvider({ children }: Props) {
       item?.id !==
         legacyScreenData(state?.location?.state as NavigationScreenData)?.id
     ) {
+      replaceEvent();
       navigateTo(item, ACTIONS.REPLACE, options);
     }
   };
@@ -275,7 +270,7 @@ export function NavigationProvider({ children }: Props) {
         route.includes(pathname)
       );
 
-      return pathnameStackIndex > 1;
+      return pathnameStackIndex >= 1;
     }
 
     if (!Number.isNaN(filteredStack?.length)) {
@@ -373,7 +368,18 @@ export function NavigationProvider({ children }: Props) {
     }
   };
 
-  const handleHookPresent = ({ hookPlugin, route, payload }, dispose) => {
+  const handleHookPresent = (
+    {
+      hookPlugin,
+      route,
+      payload,
+    }: {
+      hookPlugin: Hook;
+      route: string;
+      payload: object;
+    },
+    dispose
+  ) => {
     if (hookPlugin?.lastHook) {
       dispose();
     }
@@ -407,9 +413,12 @@ export function NavigationProvider({ children }: Props) {
     }
   };
 
-  const handleHookCancel = ({ hookPlugin }, dispose) => {
-    if (hookPlugin?.lastHook) {
-      dispose();
+  const handleHookCancel = ({ hookPlugin, callback }, dispose) => {
+    dispose();
+    setStartUpHooks(false);
+
+    if (callback && typeof callback === "function") {
+      callback();
     }
 
     if (hookPlugin.isModalHook()) {
@@ -475,7 +484,13 @@ export function NavigationProvider({ children }: Props) {
     return false;
   }, [pluginConfigurations, plugins, rivers]);
 
-  const [startUpHooks, setStartUpHooks] = React.useState(checkStartUpHooks);
+  const [startUpHooks, _setStartUpHooks] = React.useState(checkStartUpHooks);
+  const currentStartUpHooks = React.useRef(checkStartUpHooks);
+
+  const setStartUpHooks = (value) => {
+    _setStartUpHooks(value);
+    currentStartUpHooks.current = value;
+  };
 
   const handleHookComplete =
     (navigationAction, targetRoute, { screen, options }) =>
@@ -483,7 +498,7 @@ export function NavigationProvider({ children }: Props) {
       if (hookPlugin?.lastHook) {
         dispose({ disposeAll: true });
 
-        startUpHooks && setStartUpHooks(false);
+        startUpHooks.current && setStartUpHooks(false);
       }
 
       dispatch(
@@ -592,10 +607,15 @@ export function NavigationProvider({ children }: Props) {
           targetScreen: screen,
           rivers,
           plugins,
-          startUpHooksData: { startUpHooks, setStartUpHooks },
+          startUpHooksData: {
+            startUpHooks: currentStartUpHooks.current,
+            setStartUpHooks,
+          },
         };
 
-        HooksManager(hooksOptions)
+        const hooksManager = HooksManager(hooksOptions);
+
+        hooksManager.subscriber
           .on(HOOKS_EVENTS.PRESENT_SCREEN_HOOK, handleHookPresent)
           .on(HOOKS_EVENTS.ERROR, handleHookError)
           .on(HOOKS_EVENTS.CANCEL, handleHookCancel)
@@ -606,8 +626,9 @@ export function NavigationProvider({ children }: Props) {
               screen,
               options,
             })
-          )
-          .handleHooks(legacyScreenData({ entry, screen }));
+          );
+
+        hooksManager.handleHooks(legacyScreenData({ entry, screen }));
       } else {
         dispatch(navigationAction(targetRoute, { screen, entry, options }));
       }
@@ -665,15 +686,15 @@ export function NavigationProvider({ children }: Props) {
 
   // It will only work for the regular stack and pathname, It won't work for modals
   // TODO: integrate video/hook and regular modal into stack.
-  const getStackForPathname = React.useCallback((pathname: string):
-    | NavigationScreenState
-    | undefined => {
-    const stack = [...stateRef?.current?.stack?.mainStack];
+  const getStackForPathname = (
+    pathname: string
+  ): NavigationScreenState | undefined => {
+    const stack = [...state?.stack?.mainStack];
 
     const currentStack = stack.find(({ route }) => route === pathname);
 
     return currentStack;
-  }, []);
+  };
 
   const navigator: QuickBrickAppNavigator = React.useMemo(
     () => ({
@@ -717,6 +738,7 @@ export function NavigationProvider({ children }: Props) {
       replaceTop,
       getStack,
       getStackForPathname,
+      mainStack: state.stack.mainStack,
     }),
     [rivers, plugins, JSON.stringify(state), appLaunched, startUpHooks]
   );

package/App/NavigationProvider/__tests__/navigationProvider.test.tsx

@@ -5,6 +5,9 @@ import { createStore } from "redux";
 import { Provider } from "react-redux";
 
 import { useNavigation } from "@applicaster/zapp-react-native-utils/reactHooks/navigation";
+import { getByTestId } from "@applicaster/zapp-react-native-utils/testUtils/getByTestId";
+// @TODO @szhigunov to complete the migration to @testing-library/react-native
+// import { act, waitFor } from "@testing-library/react-native";
 
 const rivers = {
   A1234: {
@@ -151,37 +154,58 @@ const failingHook = {
   name: "failing hook",
 };
 
+// const store = {
+//   rivers,
+//   plugins: [hookPlugin, hookPlugin2, failingHook],
+// };
+
+const store = createStore(() => ({
+  appData: { layoutVersion: "v1" },
+  contentTypes: null,
+  rivers,
+  plugins: [hookPlugin, hookPlugin2, failingHook],
+  appState: { appLaunched: true },
+}));
+
 const ContextConsumer = () => {
   const { state, ...navigator } = useNavigation();
 
-  const renderHook = () => {
-    if (navigator.currentRoute.includes("/hooks/")) {
-      const { hookPlugin, payload, callback } = navigator.screenData;
-      const Component = hookPlugin.module.Component;
+  let renderHook;
 
-      return <Component payload={payload} callback={callback} />;
-    }
+  if (navigator.currentRoute.includes("/hooks/")) {
+    const { hookPlugin, payload, callback } = navigator.screenData;
+    const Component = hookPlugin.module.Component;
 
-    return null;
-  };
+    renderHook = <Component payload={payload} callback={callback} />;
+  }
+
+  renderHook = null;
+
+  const WrapperView: any = "View";
 
   return (
-    <View navigator={navigator} state={state}>
-      {renderHook()}
-    </View>
+    <WrapperView testID="WrapperView" navigator={navigator} state={state}>
+      {renderHook}
+    </WrapperView>
   );
 };
 
-const store = createStore(() => ({
-  appData: { layoutVersion: "v1" },
-  contentTypes: null,
-  rivers,
-  plugins: [hookPlugin, hookPlugin2, failingHook],
-  appState: { appLaunched: true },
-}));
-
 const consoleSpy = jest.spyOn(console, "error").mockImplementation(() => {});
 
+// const renderContextConsumer = (_store = store) => {
+//   return renderWithProviders(<ContextConsumer />, _store);
+// };
+
+const renderContextConsumer = (_store = store) => {
+  return create(
+    <Provider store={store}>
+      <NavigationProvider>
+        <ContextConsumer />
+      </NavigationProvider>
+    </Provider>
+  );
+};
+
 afterAll(() => {
   consoleSpy.mockReset();
 });
@@ -189,86 +213,39 @@ afterAll(() => {
 describe("<NavigationProvider />", () => {
   let wrapper;
 
-  it("renders correctly", () => {
+  it("navigator contains expected properties", () => {
     act(() => {
-      wrapper = create(
-        <Provider store={store}>
-          <NavigationProvider>
-            <ContextConsumer />
-          </NavigationProvider>
-        </Provider>
-      );
+      wrapper = renderContextConsumer();
     });
 
-    expect(wrapper.toJSON()).toMatchSnapshot();
-  });
+    const view = getByTestId(wrapper, "WrapperView");
 
-  describe("navigator Properties", () => {
-    act(() => {
-      wrapper = create(
-        <Provider store={store}>
-          <NavigationProvider>
-            <ContextConsumer />
-          </NavigationProvider>
-        </Provider>
-      );
-    });
-
-    const view = wrapper.root.findByType(View);
     const { navigator } = view.props;
+    expect(navigator.activeRiver).toEqual(rivers.B4567);
+    expect(navigator.currentRoute).toEqual("/river/B4567");
+    expect(navigator.previousAction).toEqual("REPLACE");
 
-    it("exposes the active river", () => {
-      expect(navigator.activeRiver).toEqual(rivers.B4567);
-    });
-
-    it("exposes the currentRoute", () => {
-      expect(navigator.currentRoute).toEqual("/river/B4567");
+    expect(navigator.screenData).toEqual({
+      ...rivers.B4567,
+      targetScreen: rivers.B4567,
     });
 
-    it("exposes the previous action", () => {
-      expect(navigator.previousAction).toEqual("REPLACE");
-    });
-
-    it("exposes the screen data", () => {
-      expect(navigator.screenData).toEqual(rivers.B4567);
-    });
-
-    it("exposes a routeData function", () => {
-      const consoleSpy = jest
-        .spyOn(console, "warn")
-        .mockImplementation(jest.fn());
-
-      expect(navigator.routeData()).toEqual(rivers.B4567);
-      consoleSpy.mockRestore();
-    });
-
-    it("warns in the console when calling navigator.routeData()", () => {
-      const consoleSpy = jest
-        .spyOn(console, "warn")
-        .mockImplementation(jest.fn());
-
-      navigator.routeData();
-      expect(consoleSpy.mock.calls).toMatchSnapshot();
-
-      consoleSpy.mockRestore();
+    expect(navigator.routeData()).toEqual({
+      ...rivers.B4567,
+      targetScreen: rivers.B4567,
     });
   });
 
   describe("pushing a new route", () => {
     act(() => {
-      wrapper = create(
-        <Provider store={store}>
-          <NavigationProvider>
-            <ContextConsumer />
-          </NavigationProvider>
-        </Provider>
-      );
+      wrapper = renderContextConsumer();
     });
 
-    const view = wrapper.root.findByType(View);
+    it("sets the data to that route", async () => {
+      const view = getByTestId(wrapper, "WrapperView");
 
-    it("sets the data to that route", () => {
       act(() => view.props.navigator.push(rivers.A1234));
+
       expect(view.props.navigator).toHaveProperty("activeRiver", rivers.A1234);
 
       expect(view.props.navigator).toHaveProperty(
@@ -276,23 +253,21 @@ describe("<NavigationProvider />", () => {
         "/river/B4567/river/A1234"
       );
 
-      expect(view.props.navigator.screenData).toEqual(rivers.A1234);
-      expect(view.props.state.stack.mainStack).toBeArrayOfSize(2);
+      expect(view.props.navigator.screenData).toEqual({
+        ...rivers.A1234,
+        targetScreen: rivers.A1234,
+      });
+
+      expect(view.props.state.stack.mainStack).toHaveLength(2);
     });
   });
 
   describe("replacing a current route", () => {
     act(() => {
-      wrapper = create(
-        <Provider store={store}>
-          <NavigationProvider>
-            <ContextConsumer />
-          </NavigationProvider>
-        </Provider>
-      );
+      wrapper = renderContextConsumer();
     });
 
-    const view = wrapper.root.findByType(View);
+    const view = getByTestId(wrapper, "WrapperView");
 
     it("sets the data to that route", () => {
       act(() => view.props.navigator.replace(rivers.A1234));
@@ -303,24 +278,22 @@ describe("<NavigationProvider />", () => {
         "/river/A1234"
       );
 
-      expect(view.props.navigator.screenData).toEqual(rivers.A1234);
-      expect(view.props.state.stack.mainStack).toBeArrayOfSize(1);
+      expect(view.props.navigator.screenData).toEqual({
+        ...rivers.A1234,
+        targetScreen: rivers.A1234,
+      });
+
+      expect(view.props.state.stack.mainStack).toHaveLength(1);
     });
   });
 
   describe("going back", () => {
     describe("when there is no previous route", () => {
       act(() => {
-        wrapper = create(
-          <Provider store={store}>
-            <NavigationProvider>
-              <ContextConsumer />
-            </NavigationProvider>
-          </Provider>
-        );
+        wrapper = renderContextConsumer();
       });
 
-      const view = wrapper.root.findByType(View);
+      const view = getByTestId(wrapper, "WrapperView");
 
       it("returns false when calling canGoBack", () => {
         let result;
@@ -345,23 +318,21 @@ describe("<NavigationProvider />", () => {
           "/river/B4567"
         );
 
-        expect(view.props.navigator.screenData).toEqual(rivers.B4567);
-        expect(view.props.state.stack.mainStack).toBeArrayOfSize(1);
+        expect(view.props.navigator.screenData).toEqual({
+          ...rivers.B4567,
+          targetScreen: rivers.B4567,
+        });
+
+        expect(view.props.state.stack.mainStack).toHaveLength(1);
       });
     });
 
     describe("when there is one route and we're going in a hook", () => {
       act(() => {
-        wrapper = create(
-          <Provider store={store}>
-            <NavigationProvider>
-              <ContextConsumer />
-            </NavigationProvider>
-          </Provider>
-        );
+        wrapper = renderContextConsumer();
       });
 
-      const view = wrapper.root.findByType(View);
+      const view = getByTestId(wrapper, "WrapperView");
       act(() => view.props.navigator.replace(rivers.A1234));
 
       it("returns true when calling goBack()", () => {
@@ -387,25 +358,20 @@ describe("<NavigationProvider />", () => {
         );
 
         expect(view.props.navigator.screenData).toEqual(rivers.A1234);
-        expect(view.props.state.stack.mainStack).toBeArrayOfSize(1);
+        expect(view.props.state.stack.mainStack).toHaveLength(1);
       });
     });
 
     describe("when there is a previous route", () => {
       act(() => {
-        wrapper = create(
-          <Provider store={store}>
-            <NavigationProvider>
-              <ContextConsumer />
-            </NavigationProvider>
-          </Provider>
-        );
+        wrapper = renderContextConsumer();
       });
 
-      const view = wrapper.root.findByType(View);
+      const view = getByTestId(wrapper, "WrapperView");
 
       act(() => view.props.navigator.push(rivers.A1234));
-      act(() => view.props.navigator.push(rivers.B4567));
+
+      act(() => view.props.navigator.push({ ...rivers.B4567, id: "B4567-1" }));
 
       it("returns true when calling canGoBack", () => {
         let result;
@@ -414,11 +380,27 @@ describe("<NavigationProvider />", () => {
           result = view.props.navigator.canGoBack();
         });
 
-        expect(result).toBe(true);
+        expect(result).toBeTruthy();
+      });
+
+      it("returns true when calling canGoBack(false, pathname)", () => {
+        let result;
+
+        act(() => {
+          result = view.props.navigator.canGoBack(false, "B4567-1");
+        });
+
+        expect(result).toBeTruthy();
+
+        act(() => {
+          result = view.props.navigator.canGoBack(false, "A1234");
+        });
+
+        expect(result).toBeTruthy();
       });
 
       it("returns to the previous route", () => {
-        expect(view.props.state.stack.mainStack).toBeArrayOfSize(3);
+        expect(view.props.state.stack.mainStack).toHaveLength(3);
 
         act(() => view.props.navigator.goBack());
 
@@ -432,141 +414,71 @@ describe("<NavigationProvider />", () => {
           "/river/B4567/river/A1234"
         );
 
-        expect(view.props.navigator.screenData).toEqual(rivers.A1234);
-        expect(view.props.state.stack.mainStack).toBeArrayOfSize(2);
+        expect(view.props.navigator.screenData).toEqual({
+          ...rivers.A1234,
+          targetScreen: rivers.A1234,
+        });
+
+        expect(view.props.state.stack.mainStack).toHaveLength(2);
       });
     });
 
     describe("when there is a previous route and there are hooks", () => {
       act(() => {
-        wrapper = create(
-          <Provider store={store}>
-            <NavigationProvider>
-              <ContextConsumer />
-            </NavigationProvider>
-          </Provider>
-        );
+        wrapper = renderContextConsumer();
       });
 
-      const view = wrapper.root.findByType(View);
+      const view = getByTestId(wrapper, "WrapperView");
       act(() => view.props.navigator.push(rivers.A1234));
       act(() => view.props.navigator.push(rivers.C0987));
 
       it("returns to the previous entry in the stack which is not a hook", () => {
-        expect(view.props.state.stack.mainStack).toBeArrayOfSize(5);
+        expect(view.props.state.stack.mainStack).toHaveLength(3);
 
         expect(view.props.navigator).toHaveProperty(
           "currentRoute",
           "/river/B4567/river/A1234/river/C0987"
         );
 
-        expect(view.props.navigator.screenData).toEqual(rivers.C0987);
+        expect(view.props.navigator.screenData).toMatchObject({
+          ...rivers.C0987,
+          targetScreen: rivers.C0987,
+        });
 
         act(() => view.props.navigator.goBack());
 
-        expect(view.props.state.stack.mainStack).toBeArrayOfSize(2);
+        expect(view.props.state.stack.mainStack).toHaveLength(2);
 
         expect(view.props.navigator).toHaveProperty(
           "currentRoute",
           "/river/B4567/river/A1234"
         );
 
-        expect(view.props.navigator.screenData).toEqual(rivers.A1234);
+        expect(view.props.navigator.screenData).toMatchObject({
+          ...rivers.A1234,
+          targetScreen: rivers.A1234,
+        });
       });
     });
   });
 
-  describe("when changing navBar visibility", () => {
-    act(() => {
-      wrapper = create(
-        <Provider store={store}>
-          <NavigationProvider>
-            <ContextConsumer />
-          </NavigationProvider>
-        </Provider>
-      );
-    });
-
-    const view = wrapper.root.findByType(View);
-
-    it("updates the navbar visibility attribute", () => {
-      act(() => view.props.navigator.hideNavBar());
-      expect(view.props.state.options.navBar.visible).toBe(false);
-      expect(view.props.navigator.isNavBarVisible).toBe(false);
-    });
-  });
-
-  describe("set the navbar title", () => {
-    act(() => {
-      wrapper = create(
-        <Provider store={store}>
-          <NavigationProvider>
-            <ContextConsumer />
-          </NavigationProvider>
-        </Provider>
-      );
-    });
-
-    const view = wrapper.root.findByType(View);
-
-    it("updates the navbar title attribute", () => {
-      const screenTitle = "Screen Title";
-      act(() => view.props.navigator.setScreenTitle(screenTitle));
-      expect(view.props.state.options.navBar.title).toBe(screenTitle);
-      expect(view.props.navigator.screenTitle).toBe(screenTitle);
-    });
-  });
-
-  describe("set the navbar summary", () => {
-    act(() => {
-      wrapper = create(
-        <Provider store={store}>
-          <NavigationProvider>
-            <ContextConsumer />
-          </NavigationProvider>
-        </Provider>
-      );
-    });
-
-    const view = wrapper.root.findByType(View);
-
-    it("updates the navbar summary attribute", () => {
-      const screenSummary = "Screen Summary";
-      act(() => view.props.navigator.setScreenSummary(screenSummary));
-      expect(view.props.state.options.navBar.summary).toBe(screenSummary);
-      expect(view.props.navigator.screenSummary).toBe(screenSummary);
-    });
-  });
-
-  describe.only("modal feature", () => {
+  describe("modal feature", () => {
     it("sets the modalState when opening a modal", () => {
       act(() => {
-        wrapper = create(
-          <Provider store={store}>
-            <NavigationProvider>
-              <ContextConsumer />
-            </NavigationProvider>
-          </Provider>
-        );
+        wrapper = renderContextConsumer();
       });
 
-      const view = wrapper.root.findByType(View);
+      const view = getByTestId(wrapper, "WrapperView");
       act(() => view.props.navigator.openModal({ item: rivers.A1234 }));
       expect(view.props.navigator.get).toMatchSnapshot();
     });
 
     it("removes the visible flag when dismissing a modal", () => {
       act(() => {
-        wrapper = create(
-          <Provider store={store}>
-            <NavigationProvider>
-              <ContextConsumer />
-            </NavigationProvider>
-          </Provider>
-        );
+        wrapper = renderContextConsumer();
       });
 
-      const view = wrapper.root.findByType(View);
+      const view = getByTestId(wrapper, "WrapperView");
       act(() => view.props.navigator.openModal({ item: rivers.A1234 }));
       expect(view.props.navigator.getModalState()).toMatchSnapshot();
       act(() => view.props.navigator.dismissModal());
@@ -576,16 +488,10 @@ describe("<NavigationProvider />", () => {
 
     it("should reset state to initial when dismissing a modal ( closeModalBottomSheet ) opened by openModalBottomSheet", () => {
       act(() => {
-        wrapper = create(
-          <Provider store={store}>
-            <NavigationProvider>
-              <ContextConsumer />
-            </NavigationProvider>
-          </Provider>
-        );
+        wrapper = renderContextConsumer();
       });
 
-      const view = wrapper.root.findByType(View);
+      const view = getByTestId(wrapper, "WrapperView");
 
       act(() =>
         view.props.navigator.openModalBottomSheet({

package/App/NetworkStatusProvider/utils.ts

@@ -3,7 +3,10 @@ import * as R from "ramda";
 
 import { NetInfoState } from "@react-native-community/netinfo";
 import { NETWORK_HOOKS_KEYS } from "./const";
-import { platformSelect } from "@applicaster/zapp-react-native-utils/reactUtils";
+import {
+  isVizioPlatform,
+  platformSelect,
+} from "@applicaster/zapp-react-native-utils/reactUtils";
 import { Platform } from "react-native";
 
 type ConnectionStatus = boolean;
@@ -203,6 +206,18 @@ export const subscribeToDeviceNetworkChanges = (callback: NetCb) => {
     registeredListener = webapis.network.addNetworkStateChangeListener(
       samsungNetworkChangeHandler(callback)
     );
+
+    return;
+  }
+
+  if (isVizioPlatform()) {
+    window.addEventListener("online", function () {
+      callback({ online: true });
+    });
+
+    window.addEventListener("offline", function () {
+      callback({ online: false });
+    });
   }
 };
 
@@ -214,5 +229,12 @@ export const subscribeToDeviceNetworkChanges = (callback: NetCb) => {
 export const unsubscribeToDeviceNetworkChanges = (id = registeredListener) => {
   if (hasTizenAPIs()) {
     webapis.network.removeNetworkStateChangeListener(id);
+
+    return;
+  }
+
+  if (isVizioPlatform()) {
+    window.removeEventListener("online", function () {});
+    window.removeEventListener("offline", function () {});
   }
 };

package/App/RouteManager/__tests__/__snapshots__/routeManager.test.js.snap

@@ -13,7 +13,6 @@ exports[`<RouteManager /> renders correctly 1`] = `
   }
 >
   <View
-    comp="river"
     testID="river"
   />
 </RNCSafeAreaProvider>

package/App/index.tsx

@@ -1,8 +1,8 @@
 import * as React from "react";
 import * as R from "ramda";
 import {
-  SafeAreaProvider,
   initialWindowMetrics,
+  SafeAreaProvider,
 } from "react-native-safe-area-context";
 
 import { ErrorBoundary } from "./ErrorBoundary";
@@ -27,6 +27,7 @@ import {
 import { withURLSchemeContextProvider } from "./DeepLinking/URLSchemeListener/URLSchemeContextProvider";
 import { appRemoteDataLoader } from "./appRemoteDataLoader";
 import { withActionExecutor } from "@applicaster/zapp-react-native-utils/actionsExecutor/ActionExecutorContext";
+import { withScreenDataContextProvider } from "@applicaster/zapp-react-native-ui-components/Contexts/ScreenDataContext";
 
 /**
  * @typedef {Object} AppOptions
@@ -99,6 +100,7 @@ export function createQuickBrickApp(options): ZappApp {
     withURLSchemeContextProvider,
     withNetworkStatusProvider,
     withNavigationProvider,
+    withScreenDataContextProvider,
     ZappPipesSearchContext.withProvider,
     InteractionManager,
     withActionExecutor,

package/const/index.ts

@@ -34,3 +34,7 @@ export const requiredSessionKeys = [
   "userAgent",
   "version_name",
 ];
+
+export const QUICK_BRICK_NAVBAR = "quick-brick-navbar";
+
+export const QUICK_BRICK_CONTENT = "quick-brick-content";

package/createZappApp/index.tsx

@@ -7,6 +7,10 @@
 import * as React from "react";
 import * as R from "ramda";
 import { Dimensions } from "react-native";
+import {
+  isNotEmpty,
+  isNotNil,
+} from "@applicaster/zapp-react-native-utils/reactUtils/helpers";
 
 import {
   createZappReduxStore,
@@ -27,6 +31,9 @@ import { remoteDataLoader } from "../App/appRemoteDataLoader/remoteDataLoader";
 import * as QuickBrickManager from "@applicaster/zapp-react-native-bridge/QuickBrick";
 import { isAndroidPlatform } from "@applicaster/zapp-react-native-utils/reactUtils";
 
+const isNotEmptyObject = (value: unknown) =>
+  isNotNil(value) && R.is(Object, value) && isNotEmpty(value);
+
 /**
  * Creates a Zapp App with the provided options
  * @export
@@ -55,8 +62,8 @@ export function createZappApp(zappAppConfig: ZappAppConfig) {
 
   const useTabletLayout =
     isTablet({ width, height }, width > height ? "landscape" : "portrait") &&
-    tabletLayout &&
-    tabletCellStyles;
+    isNotEmptyObject(tabletLayout) &&
+    isNotEmptyObject(tabletCellStyles);
 
   function fetchPluginConfiguration(plugin) {
     const configuration = R.compose(

package/defaults/index.js

@@ -1,5 +1,3 @@
-// @flow
-
 /**
  * quick-brick-core module
  * @module @applicaster/quick-brick-core/defaults

package/package.json

@@ -1,12 +1,11 @@
 {
   "name": "@applicaster/quick-brick-core",
-  "version": "12.0.0-alpha.5040434238",
+  "version": "12.0.0-alpha.5050333009",
   "description": "Core package for Applicaster's Quick Brick App",
   "main": "index.js",
   "types": "index.d.ts",
   "scripts": {
     "test": "jest -w=2",
-    "flow-types": "node_modules/.bin/flow-copy-source -v -i '**/__tests__**' . .",
     "build": "echo 'skipping ts build for core packages'"
   },
   "repository": {
@@ -30,13 +29,13 @@
   },
   "homepage": "https://github.com/applicaster/quickbrick#readme",
   "dependencies": {
-    "@applicaster/applicaster-types": "12.0.0-alpha.5040434238",
-    "@applicaster/quick-brick-core-plugins": "12.0.0-alpha.5040434238",
-    "@applicaster/zapp-pipes-v2-client": "12.0.0-alpha.5040434238",
-    "@applicaster/zapp-react-native-bridge": "12.0.0-alpha.5040434238",
-    "@applicaster/zapp-react-native-redux": "12.0.0-alpha.5040434238",
-    "@applicaster/zapp-react-native-ui-components": "12.0.0-alpha.5040434238",
-    "@applicaster/zapp-react-native-utils": "12.0.0-alpha.5040434238",
+    "@applicaster/applicaster-types": "12.0.0-alpha.5050333009",
+    "@applicaster/quick-brick-core-plugins": "12.0.0-alpha.5050333009",
+    "@applicaster/zapp-pipes-v2-client": "12.0.0-alpha.5050333009",
+    "@applicaster/zapp-react-native-bridge": "12.0.0-alpha.5050333009",
+    "@applicaster/zapp-react-native-redux": "12.0.0-alpha.5050333009",
+    "@applicaster/zapp-react-native-ui-components": "12.0.0-alpha.5050333009",
+    "@applicaster/zapp-react-native-utils": "12.0.0-alpha.5050333009",
     "atob": "^2.1.2",
     "axios": "^0.28.0",
     "btoa": "^1.2.1",