Escape nullable policies

cartridge-gg · Aug 20, 2024 · 674dd38 · 674dd38
1 parent 5318c6b
commit 674dd38
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 5 deletions.
diff --git a/examples/starknet-react-next/src/components/providers/StarknetProvider.tsx b/examples/starknet-react-next/src/components/providers/StarknetProvider.tsx
@@ -60,7 +60,6 @@ const cartridge = new CartridgeConnector({
           "cartridge-starknet-react-next",
           "keychain",
         ),
-
   rpc: process.env.NEXT_PUBLIC_RPC_SEPOLIA,
   paymaster: {
     caller: shortString.encodeShortString("ANY_CALLER"),

diff --git a/packages/keychain/src/Policies.tsx b/packages/keychain/src/Policies.tsx
@@ -48,7 +48,7 @@ export function Policies({
             // The container already set border radius (for top & bottom), but we
             // set the bottom radius for the last item here because for certain
             // browsers' scrolling behaviour (eg Firefox) just to make it look nicer.
-            borderBottomRadius={i === policies.length - 1 ? "base" : "none"}
+            borderBottomRadius={i === policies?.length - 1 ? "base" : "none"}
           >
             {({ isExpanded }) => (
               <>

diff --git a/packages/keychain/src/components/connect/Login.tsx b/packages/keychain/src/components/connect/Login.tsx
@@ -86,7 +86,7 @@ function Form({
           await doLogin(usernameField.value, credentialId);
           break;
         case LoginMode.Controller:
-          if (policies?.length === 0) {
+          if (!policies?.length) {
             break;
           }
 

diff --git a/packages/keychain/src/components/layout/Footer/index.tsx b/packages/keychain/src/components/layout/Footer/index.tsx
@@ -32,7 +32,7 @@ export function Footer({
   const isExpandable = useMemo(
     () =>
       !!origin &&
-      !!policies.length &&
+      !!policies?.length &&
       variant === "connect" &&
       !isSignup &&
       !hideTxSummary,
@@ -100,7 +100,7 @@ export function Footer({
           onClick={footer.onToggle}
           _hover={{ cursor: "pointer" }}
         >
-          {!hideTxSummary && !!policies.length && (
+          {!hideTxSummary && !!policies?.length && (
             <TransactionSummary
               isSlot={isSlot}
               createSession={createSession}