Fix inapp wallets showing up in all wallets list

Author: joaquim-verges

.changeset/tangy-otters-invite.md

@@ -0,0 +1,5 @@
+---
+"thirdweb": patch
+---
+
+Fix inapp wallets showing up in all wallets list

packages/thirdweb/src/react/native/ui/connect/ExternalWalletsList.tsx

@@ -100,6 +100,7 @@ export function AllWalletsList(
       .filter(
         (info) => !externalWallets.find((wallet) => wallet.id === info.id),
       )
+      .filter((info) => info.id !== "inApp" && info.id !== "embedded" && info.id !== "smart")
       .filter((info) => info.hasMobileSupport);
 
     const fuse = new Fuse(filteredWallets, {

packages/thirdweb/src/react/web/ui/ConnectWallet/Modal/AllWalletsUI.tsx

@@ -39,9 +39,16 @@ function AllWalletsUI(props: {
   const setSelectionData = useSetSelectionData();
 
   const walletList = useMemo(() => {
-    return walletInfos.filter((wallet) => {
-      return props.specifiedWallets.findIndex((x) => x.id === wallet.id) === -1;
-    });
+    return walletInfos
+      .filter((wallet) => {
+        return (
+          props.specifiedWallets.findIndex((x) => x.id === wallet.id) === -1
+        );
+      })
+      .filter(
+        (info) =>
+          info.id !== "inApp" && info.id !== "embedded" && info.id !== "smart"
+      );
   }, [props.specifiedWallets]);
 
   const fuseInstance = useMemo(() => {
@@ -183,15 +190,15 @@ function AllWalletsUI(props: {
   );
 }
 
-const StyledMagnifyingGlassIcon = /* @__PURE__ */ styled(MagnifyingGlassIcon)(
-  (_) => {
-    const theme = useCustomTheme();
-    return {
-      color: theme.colors.secondaryText,
-      left: spacing.sm,
-      position: "absolute",
-    };
-  },
-);
+const StyledMagnifyingGlassIcon = /* @__PURE__ */ styled(MagnifyingGlassIcon)((
+  _
+) => {
+  const theme = useCustomTheme();
+  return {
+    color: theme.colors.secondaryText,
+    left: spacing.sm,
+    position: "absolute",
+  };
+});
 
 export default AllWalletsUI;