@applicaster/quick-brick-core 14.0.0-alpha.3259144304 → 14.0.0-alpha.3497255033

package/App/ActionSetters/index.ts

@@ -12,7 +12,6 @@ const debugActions = (actions) =>
     message: `useSubscribeToActionProviders - available actions count: ${
       Object.keys(actions).length
     }`,
-    data: { actions },
   });
 
 const warnUndefinedContext = (context) =>

package/App/ActionsProvider/index.js

@@ -1,7 +1,7 @@
 import * as React from "react";
 import * as R from "ramda";
 
-import { connectToStore } from "@applicaster/zapp-react-native-redux";
+import { connectToStore } from "@applicaster/zapp-react-native-redux/utils/connectToStore";
 
 import { ActionsProvider as ActionsProviderComponent } from "./ActionsProvider";
 

package/App/AppStateDecorator/index.tsx

@@ -2,9 +2,9 @@ import * as React from "react";
 import { AppState, AppStateStatus } from "react-native";
 import * as R from "ramda";
 
-import { connectToStore } from "@applicaster/zapp-react-native-redux";
+import { connectToStore } from "@applicaster/zapp-react-native-redux/utils/connectToStore";
 import { setAppState } from "@applicaster/zapp-react-native-redux/appState";
-import { Dispatch } from "redux";
+import { Dispatch } from "@reduxjs/toolkit";
 
 type Props = {
   Component: () => React.ComponentType<any>;
@@ -19,6 +19,7 @@ function withAppManagerDecorator(
     currentState: AppStateStatus;
     isAvailable: boolean;
     plugins: QuickBrickPlugin[];
+    appStateListener: ReturnType<typeof AppState.addEventListener>;
 
     constructor(props: Props) {
       super(props);
@@ -30,11 +31,15 @@ function withAppManagerDecorator(
 
     componentDidMount() {
       this.plugins = this.getAppStatePlugins();
-      AppState.addEventListener("change", this.onChange);
+
+      this.appStateListener = AppState.addEventListener(
+        "change",
+        this.onChange
+      );
     }
 
     componentWillUnmount() {
-      AppState.removeEventListener("change", this.onChange);
+      this.appStateListener.remove();
     }
 
     dispatchAction(nextState: AppStateStatus, eventType) {

package/App/DeepLinking/URLSchemeHandler/SchemeHandlerHooks/__tests__/usePresentSchemeHandler.test.tsx

@@ -4,7 +4,6 @@ import { renderHook } from "@testing-library/react-native";
 import { sessionStorage } from "@applicaster/zapp-react-native-bridge/ZappStorage/SessionStorage";
 import * as helpers from "../../../helpers";
 import * as feedLoader from "@applicaster/zapp-react-native-utils/reactHooks/feed/useFeedLoader";
-import * as useNavigationHooks from "@applicaster/zapp-react-native-utils/reactHooks/navigation/useNavigation";
 import { WrappedWithProviders } from "@applicaster/zapp-react-native-utils/testUtils";
 import { usePresentSchemeHandler } from "../usePresentSchemeHandler";
 
@@ -16,15 +15,20 @@ const rivers = {
   },
 };
 
-const navigator = {
+const mock_navigator = {
   currentRoute: "/river/A1234",
   replace: jest.fn(),
   goHome: jest.fn(),
 } as any;
 
-const mockStore = configureStore();
+jest.mock(
+  "@applicaster/zapp-react-native-utils/reactHooks/navigation/useNavigation",
+  () => ({
+    useNavigation: jest.fn(() => mock_navigator),
+  })
+);
 
-jest.spyOn(useNavigationHooks, "useNavigation").mockReturnValue(navigator);
+const mockStore = configureStore();
 
 const helperSpy = jest.spyOn(helpers, "handlePresentNavigation");
 const feedLoaderSpy = jest.spyOn(feedLoader, "useFeedLoader");
@@ -33,6 +37,7 @@ const onFinish = jest.fn((cb) => {
   cb();
 });
 
+const navigator = mock_navigator;
 jest.useFakeTimers();
 
 describe("usePresentSchemeHandler", () => {
@@ -210,6 +215,8 @@ describe("usePresentSchemeHandler", () => {
           actions?.find(R.propEq("type", "SET_APP_NOT_READY"))
         ).toBeDefined();
 
+        expect(actions?.find(R.propEq("type", "SET_APP_READY"))).toBeDefined();
+
         expect(navigator.replace).toBeCalledWith({});
       };
     });

package/App/DeepLinking/URLSchemeHandler/SchemeHandlerHooks/useOpenSchemeHandler/index.ts

@@ -1,4 +1,8 @@
-import { usePickFromState } from "@applicaster/zapp-react-native-redux/hooks";
+import {
+  useAppSelector,
+  selectContentTypes,
+  selectRivers,
+} from "@applicaster/zapp-react-native-redux";
 import { ZappPipesEntryContext } from "@applicaster/zapp-react-native-ui-components/Contexts";
 import { useNavigation } from "@applicaster/zapp-react-native-utils/reactHooks/navigation";
 import { useEffect } from "react";
@@ -97,7 +101,8 @@ async function handleQuery({
 }
 
 export function useOpenSchemeHandler({ query, url, onFinish }) {
-  const { rivers, contentTypes } = usePickFromState(["rivers", "contentTypes"]);
+  const rivers = useAppSelector(selectRivers);
+  const contentTypes = useAppSelector(selectContentTypes);
   const navigator = useNavigation();
   const homeId = getHomeScreenId(rivers);
   const homeRoute = `/river/${homeId}`;

package/App/DeepLinking/URLSchemeHandler/SchemeHandlerHooks/usePresentSchemeHandler.ts

@@ -1,15 +1,17 @@
-import { useActions } from "@applicaster/zapp-react-native-redux/hooks";
+import {
+  loadAppContextData,
+  useAppDispatch,
+  useActions,
+} from "@applicaster/zapp-react-native-redux";
 import * as R from "ramda";
 import * as React from "react";
 import axios from "axios";
-import { useDispatch } from "react-redux";
 
 import { useNavigation } from "@applicaster/zapp-react-native-utils/reactHooks";
 
 import { useFeedLoader } from "@applicaster/zapp-react-native-utils/reactHooks/feed/useFeedLoader";
 
 import { sessionStorage } from "@applicaster/zapp-react-native-bridge/ZappStorage/SessionStorage";
-import { loadAppContextData } from "@applicaster/zapp-react-native-redux";
 import { noop } from "@applicaster/zapp-react-native-utils/functionUtils";
 
 import {
@@ -60,7 +62,7 @@ export function usePresentSchemeHandler({
 
   const rivers = useRivers();
   const screenRiver = findRiver({ screen_id, rivers });
-  const dispatch = useDispatch();
+  const dispatch = useAppDispatch();
 
   const actions = useActions({
     setAppReady,

package/App/DeepLinking/URLSchemeHandler/__tests__/URLSchemeHandler.test.tsx

@@ -6,16 +6,16 @@ const theme = {
   app_background_color: "white",
 };
 
-const plugins = [];
-
-jest.mock("@applicaster/zapp-react-native-utils/reactHooks/navigation");
+jest.mock(
+  "@applicaster/zapp-react-native-utils/reactHooks/navigation/useNavigation"
+);
 
 jest.mock("@applicaster/zapp-react-native-utils/theme", () => ({
   useTheme: jest.fn(() => theme),
 }));
 
 jest.mock("@applicaster/zapp-react-native-redux/hooks", () => ({
-  usePickFromState: jest.fn(() => plugins),
+  usePickFromState: jest.fn(() => ({})),
 }));
 
 const { URLSchemeHandler } = require("../URLSchemeHandler");

package/App/ErrorBoundary/index.tsx

@@ -60,6 +60,10 @@ export class ErrorBoundary extends React.Component<Props> {
     useErrorStore.getState().setError(error, info, recoverable);
   }
 
+  static getDerivedStateFromError(_error) {
+    return { hasError: true };
+  }
+
   render() {
     return <ErrorBoundaryContent>{this.props.children}</ErrorBoundaryContent>;
   }

package/App/ModalProvider/ModalBottomSheet/DraggableBottomSheet/index.tsx

@@ -52,9 +52,9 @@ export class DraggableBottomSheet extends Component<Props, ComponentState> {
   _lastScrollYValue: number;
   _lastScrollY: Animated.Value;
   _dragY: Animated.Value;
-  _reverseLastScrollY: Animated.AnimatedMultiplication;
+  _reverseLastScrollY: Animated.AnimatedInterpolation<number>;
   _translateYOffset: Animated.Value;
-  _translateY: Animated.AnimatedAddition;
+  _translateY: Animated.AnimatedInterpolation<number>;
   _snapPoints: SnapPoints;
   _prevContentHeight = 0;
 

package/App/ModalProvider/ModalBottomSheet/ModalBottomSheetFrame.tsx

@@ -42,12 +42,17 @@ const defaultBottomOffset = platformSelect({
   android: 0,
 });
 
+const SAFE_AREA_BREAKING_API_VERSION = 35;
+
 const getSheetHeight = ({ height, bottomOffset, maxHeight }) => {
   return platformSelect({
     ios: height - bottomOffset,
     default:
       (height || maxHeight) -
-      (!isAndroidVersionAtLeast(35) && isAndroidPlatform() ? bottomOffset : 0),
+      (isAndroidPlatform() &&
+      !isAndroidVersionAtLeast(SAFE_AREA_BREAKING_API_VERSION)
+        ? bottomOffset
+        : 0),
   });
 };
 

package/App/ModalProvider/ModalChildrenWrapper/index.tsx

@@ -1,13 +1,8 @@
 import React, { ReactChild } from "react";
-import { StyleSheet, ViewProps } from "react-native";
-import { GestureHandlerRootView } from "react-native-gesture-handler";
+import { View, StyleSheet, ViewProps } from "react-native";
 
 type Props = ViewProps & { children: ReactChild };
 
 export function ModalChildrenWrapper({ children }: Props) {
-  return (
-    <GestureHandlerRootView style={StyleSheet.absoluteFill}>
-      {children}
-    </GestureHandlerRootView>
-  );
+  return <View style={StyleSheet.absoluteFill}>{children}</View>;
 }

package/App/ModalProvider/ModalPresenter.tsx

@@ -1,5 +1,6 @@
 import * as React from "react";
 import { Modal, ModalProps } from "react-native";
+import { GestureHandlerRootView } from "react-native-gesture-handler";
 
 type Props = ModalProps & {
   children: React.ReactChild;
@@ -13,7 +14,7 @@ export function ModalPresenter(props: Props) {
 
   return (
     <Presenter visible={visible} {...presenterOptions}>
-      {children}
+      <GestureHandlerRootView>{children}</GestureHandlerRootView>
     </Presenter>
   );
 }

package/App/NavigationProvider/NavigationProvider.tsx

@@ -7,6 +7,7 @@ import {
   getTargetRoute,
   usesVideoModal,
 } from "@applicaster/zapp-react-native-utils/navigationUtils";
+import { last } from "@applicaster/zapp-react-native-utils/utils";
 import {
   allowedOrientationsForScreen,
   useGetScreenOrientation,
@@ -121,8 +122,20 @@ export function NavigationProvider({ children }: Props) {
     NavigationReducerState
   >(reducer, initialState, (x) => x);
 
+  const topStackRoute = last(state?.stack?.mainStack);
+
+  /** TODO: temporary variable, before we remove need for a "current".
+   There is no current as we are using stack navigator. Route data should be contextualised per screen
+   There are some cases to use a route data from top of the stack but it should be separate method.
+   */
+
+  const currentLocation = {
+    ...topStackRoute,
+    route: topStackRoute?.route || "/", // Fallback for initial state.
+  };
+
   const stateRef = React.useRef<NavigationReducerState>(state);
-  const pathname = state?.location?.pathname;
+  const pathname = currentLocation?.route; // TODO: remove. Pathname is part of the route, not a global.
   const pathnameRef = React.useRef(pathname);
 
   const { context } = React.useContext(ZappPipesEntryContext.Context);
@@ -146,6 +159,8 @@ export function NavigationProvider({ children }: Props) {
     }
   }, [pathname]);
 
+  // TODO remove.
+  // There is no concept of active river. Multiple rivers can be rendered at the same time.
   const activeRiver = React.useMemo(
     () => activeRiverSelector({ pathname, rivers }),
     [pathname, rivers]
@@ -186,6 +201,7 @@ export function NavigationProvider({ children }: Props) {
   // TODO: Move in proper place
   const modalPresenterScreenOrientation = useGetScreenOrientation(activeRiver);
 
+  // TODO: Move modal state to separate store
   const closeVideoModal = () => {
     if (navigator.modalData) {
       allowedOrientationsForScreen(modalPresenterScreenOrientation);
@@ -213,13 +229,15 @@ export function NavigationProvider({ children }: Props) {
   const isVideoModalDocked = () =>
     state?.options.videoModal.mode === "MINIMIZED";
 
+  // TODO: Remove as it's using a concept of "current" route.
+  // Route is contextualised and shouldn't be used this way.
   const routeData = () => {
     // eslint-disable-next-line no-console
     console.warn(`you are retrieving screen data from navigator.routeData()
     This function will be removed in a later version of quick brick.
     You can retrieve this data from navigator.screenData`);
 
-    return legacyScreenData(state?.location?.state as NavigationScreenData);
+    return legacyScreenData(currentLocation?.state as NavigationScreenData);
   };
 
   // TODO: remove it by refactoring back buttons handlers
@@ -262,7 +280,7 @@ export function NavigationProvider({ children }: Props) {
     if (
       targetRoute !== pathnameRef.current ||
       item?.id !==
-        legacyScreenData(state?.location?.state as NavigationScreenData)?.id
+        legacyScreenData(currentLocation?.state as NavigationScreenData)?.id
     ) {
       replaceEvent();
       navigateTo(item, ACTIONS.REPLACE, options);
@@ -688,7 +706,8 @@ export function NavigationProvider({ children }: Props) {
     }
   }, [state?.options?.videoModal?.previousMode]);
 
-  const getNestedEntry = () => state?.location?.state?.nested?.entry ?? null;
+  // TODO: remove. This shouldn't be part of navigator
+  const getNestedEntry = () => currentLocation?.state?.nested?.entry ?? null;
 
   // 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.
@@ -706,7 +725,7 @@ export function NavigationProvider({ children }: Props) {
     () => ({
       activeRiver,
       getPathname: () => pathnameRef.current, // hack use to fix issue causing broken navigation as currentRoute is unreliable
-      currentRoute: pathname,
+      currentRoute: pathname, // TODO: remove. Current route shouldn't be needed
       previousAction: lastEntrySelector(state)
         ?.action as QuickBrickNavigationActionType,
       push: pushItem,
@@ -716,14 +735,15 @@ export function NavigationProvider({ children }: Props) {
       goHome,
       canGoBack,
       goBack,
-      routeData,
+      routeData, // TODO: remove
       screenData: legacyScreenData(
-        state?.location?.state as NavigationScreenData,
+        // TODO: remove
+        currentLocation?.state as NavigationScreenData,
         plugins
       ),
-      data: state?.location?.state as NavigationScreenData,
+      data: currentLocation?.state as NavigationScreenData,
       getNestedEntry,
-      key: state?.location?.key,
+      key: currentLocation?.key, // TODO: remove
       modalData: state.stack.modal?.state ?? null,
       openModal,
       dismissModal,

package/App/NavigationProvider/navigator/__tests__/__snapshots__/reducer.test.ts.snap

@@ -13,7 +13,6 @@ exports[`ACTIONS matches snapshot 1`] = `
   "PUSH": "PUSH",
   "REPLACE": "REPLACE",
   "REPLACE_TOP": "REPLACE_TOP",
-  "SET_LOCATION": "SET_LOCATION",
   "SET_NESTED_CONTENT": "SET_NESTED_CONTENT",
   "SET_VIDEO_MODAL_ITEM": "SET_VIDEO_MODAL_ITEM",
 }
@@ -22,30 +21,6 @@ exports[`ACTIONS matches snapshot 1`] = `
 exports[`reducer when a back action is dispatched push, back, push, back does what it's supposed to do 1`] = `
 {
   "state1": {
-    "location": {
-      "key": "mocked-uuid",
-      "pathname": "/river/A1234",
-      "state": {
-        "entry": undefined,
-        "options": undefined,
-        "screen": {
-          "home": true,
-          "home_offline": true,
-          "hooks": {
-            "preload_plugins": [],
-          },
-          "id": "A1234",
-          "name": "Home",
-          "navigations": [
-            {
-              "id": "nav_id",
-            },
-          ],
-          "supports_offline": false,
-          "type": "general_content",
-        },
-      },
-    },
     "options": {
       "videoModal": {
         "item": null,
@@ -86,36 +61,6 @@ exports[`reducer when a back action is dispatched push, back, push, back does wh
     },
   },
   "state2": {
-    "location": {
-      "key": "mocked-uuid",
-      "pathname": "/river/A1234/river/B7890",
-      "state": {
-        "entry": {
-          "id": "C0987",
-          "title": "Entry",
-          "type": {
-            "value": "feed",
-          },
-        },
-        "options": undefined,
-        "screen": {
-          "home": false,
-          "home_offline": false,
-          "hooks": {
-            "preload_plugins": [],
-          },
-          "id": "B5678",
-          "name": "Screen name",
-          "navigations": [
-            {
-              "id": "nav_id",
-            },
-          ],
-          "supports_offline": false,
-          "type": "general_content",
-        },
-      },
-    },
     "options": {
       "videoModal": {
         "item": null,
@@ -187,36 +132,6 @@ exports[`reducer when a back action is dispatched push, back, push, back does wh
     },
   },
   "state3": {
-    "location": {
-      "key": "mocked-uuid",
-      "pathname": "/river/A1234/river/B7890/river/C0987",
-      "state": {
-        "entry": {
-          "id": "D1234",
-          "title": "Entry 2",
-          "type": {
-            "value": "video",
-          },
-        },
-        "options": undefined,
-        "screen": {
-          "home": false,
-          "home_offline": false,
-          "hooks": {
-            "preload_plugins": [],
-          },
-          "id": "C9090",
-          "name": "Screen 2 name",
-          "navigations": [
-            {
-              "id": "nav_id",
-            },
-          ],
-          "supports_offline": false,
-          "type": "general_content",
-        },
-      },
-    },
     "options": {
       "videoModal": {
         "item": null,
@@ -319,37 +234,6 @@ exports[`reducer when a back action is dispatched push, back, push, back does wh
     },
   },
   "state4": {
-    "location": {
-      "action": "PUSH",
-      "key": "mocked-uuid",
-      "pathname": "/river/A1234/river/B7890",
-      "state": {
-        "entry": {
-          "id": "C0987",
-          "title": "Entry",
-          "type": {
-            "value": "feed",
-          },
-        },
-        "options": undefined,
-        "screen": {
-          "home": false,
-          "home_offline": false,
-          "hooks": {
-            "preload_plugins": [],
-          },
-          "id": "B5678",
-          "name": "Screen name",
-          "navigations": [
-            {
-              "id": "nav_id",
-            },
-          ],
-          "supports_offline": false,
-          "type": "general_content",
-        },
-      },
-    },
     "options": {
       "videoModal": {
         "item": null,
@@ -421,36 +305,6 @@ exports[`reducer when a back action is dispatched push, back, push, back does wh
     },
   },
   "state5": {
-    "location": {
-      "key": "mocked-uuid",
-      "pathname": "/river/A1234/river/B7890/river/D5678",
-      "state": {
-        "entry": {
-          "id": "C0987",
-          "title": "Entry",
-          "type": {
-            "value": "feed",
-          },
-        },
-        "options": undefined,
-        "screen": {
-          "home": false,
-          "home_offline": false,
-          "hooks": {
-            "preload_plugins": [],
-          },
-          "id": "B5678",
-          "name": "Screen name",
-          "navigations": [
-            {
-              "id": "nav_id",
-            },
-          ],
-          "supports_offline": false,
-          "type": "general_content",
-        },
-      },
-    },
     "options": {
       "videoModal": {
         "item": null,
@@ -553,37 +407,6 @@ exports[`reducer when a back action is dispatched push, back, push, back does wh
     },
   },
   "state6": {
-    "location": {
-      "action": "PUSH",
-      "key": "mocked-uuid",
-      "pathname": "/river/A1234/river/B7890",
-      "state": {
-        "entry": {
-          "id": "C0987",
-          "title": "Entry",
-          "type": {
-            "value": "feed",
-          },
-        },
-        "options": undefined,
-        "screen": {
-          "home": false,
-          "home_offline": false,
-          "hooks": {
-            "preload_plugins": [],
-          },
-          "id": "B5678",
-          "name": "Screen name",
-          "navigations": [
-            {
-              "id": "nav_id",
-            },
-          ],
-          "supports_offline": false,
-          "type": "general_content",
-        },
-      },
-    },
     "options": {
       "videoModal": {
         "item": null,