@lookiero/checkout 0.6.0-beta.22 → 0.6.0-beta.24

package/dist/infrastructure/ui/routing/CheckoutAccessibilityMiddleware.js

@@ -11,7 +11,7 @@ const CheckoutAccessibilityMiddleware = ({ customerId, onNotAccessible, loader =
         if (notAccessible) {
             onNotAccessibleRef.current();
         }
-    }, [notAccessible, onNotAccessible]);
+    }, [notAccessible]);
     return accessible === undefined && [QueryStatus.IDLE, QueryStatus.LOADING].includes(status)
         ? loader
         : accessible

package/dist/infrastructure/ui/routing/Routing.d.ts

@@ -1,5 +1,5 @@
 import { I18n } from "@lookiero/i18n-react";
-import { FC } from "react";
+import React from "react";
 import { useRoutes as reactRouterUseRoutes } from "react-router-native";
 import { Customer } from "../../../projection/shared/customer";
 import { Menu, TabBar } from "../views/navigation/Navigation";
@@ -16,5 +16,11 @@ interface RoutingProps {
     readonly useRedirect: () => Record<string, string>;
     readonly useRoutes: typeof reactRouterUseRoutes;
 }
-declare const Routing: FC<RoutingProps>;
-export { Routing };
+/**
+ * Provided useRoutes is not stable (when integrated with Sentry) as it's rendering a different component tree.
+ *
+ * https://github.com/getsentry/sentry-javascript/blob/master/packages/react/src/reactrouterv6.tsx#L221
+ * (SentryRoutes is a new component after each re-render)
+ */
+declare const MemoizedRouting: React.NamedExoticComponent<RoutingProps>;
+export { MemoizedRouting as Routing };

package/dist/infrastructure/ui/routing/Routing.js

@@ -1,4 +1,4 @@
-import React, { lazy, Suspense, useMemo } from "react";
+import React, { lazy, memo, Suspense } from "react";
 import { Navigate, Outlet } from "react-router-native";
 import { Spinner } from "../../../shared/ui/components/atoms/spinner/Spinner";
 import { App } from "../views/App";
@@ -13,7 +13,7 @@ const Summary = lazy(() => import("../views/summary/Summary").then((module) => (
 const Checkout = lazy(() => import("../views/checkout/Checkout").then((module) => ({ default: module.Checkout })));
 const Feedback = lazy(() => import("../views/feedback/Feedback").then((module) => ({ default: module.Feedback })));
 const Routing = ({ basePath = "", customer, locale, I18n, menu, tabBar, getAuthToken, onI18nError, onNotAccessible, useRedirect, useRoutes, }) => {
-    const routes = useMemo(() => useRoutes([
+    const routes = useRoutes([
         {
             path: "",
             element: (React.createElement(BasePathProvider, { basePath: basePath },
@@ -51,7 +51,7 @@ const Routing = ({ basePath = "", customer, locale, I18n, menu, tabBar, getAuthT
                     children: [
                         {
                             path: Routes.CHECKOUT_PAYMENT,
-                            element: (React.createElement(CheckoutPaymentModal, { customerId: customer?.customerId, getAuthToken: getAuthToken })),
+                            element: React.createElement(CheckoutPaymentModal, { customerId: customer?.customerId, getAuthToken: getAuthToken }),
                         },
                     ],
                 },
@@ -66,20 +66,14 @@ const Routing = ({ basePath = "", customer, locale, I18n, menu, tabBar, getAuthT
                 },
             ],
         },
-    ]), [
-        I18n,
-        basePath,
-        customer?.country,
-        customer?.customerId,
-        getAuthToken,
-        locale,
-        menu,
-        onI18nError,
-        onNotAccessible,
-        tabBar,
-        useRedirect,
-        useRoutes,
     ]);
     return routes;
 };
-export { Routing };
+/**
+ * Provided useRoutes is not stable (when integrated with Sentry) as it's rendering a different component tree.
+ *
+ * https://github.com/getsentry/sentry-javascript/blob/master/packages/react/src/reactrouterv6.tsx#L221
+ * (SentryRoutes is a new component after each re-render)
+ */
+const MemoizedRouting = memo(Routing);
+export { MemoizedRouting as Routing };

package/dist/shared/logging/SentryDependencies.js

@@ -8,14 +8,6 @@ const integrations = [
     ...defaultIntegrations,
     new BrowserTracing({
         routingInstrumentation: reactRouterV6Instrumentation(useEffect, useLocation, useNavigationType, createRoutesFromChildren, matchRoutes),
-        beforeNavigate: (context) => {
-            console.log("beforeNavigate", context);
-            return context;
-        },
-        shouldCreateSpanForRequest: (url) => {
-            console.log(url);
-            return true;
-        },
     }),
 ];
 const wrap = (Component) => Component;

package/dist/shared/logging/SentryLogger.js

@@ -66,14 +66,6 @@ const sentryLogger = ({ publicKey, environment, release, project }) => {
             /webappstoolbarba\.texthelp\.com\//i,
             /metrics\.itunes\.apple\.com\.edgesuite\.net\//i,
         ],
-        beforeSend: (event) => {
-            console.log(event);
-            return event;
-        },
-        beforeSendTransaction: (event) => {
-            console.log(event);
-            return event;
-        },
     });
     const hub = new Hub(client);
     const onMount = (customerId) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lookiero/checkout",
-  "version": "0.6.0-beta.22",
+  "version": "0.6.0-beta.24",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
   "files": [