@dynamic-labs/sdk-react-core 1.4.17 → 1.4.19

package/CHANGELOG.md

@@ -1,4 +1,18 @@
 
+### [1.4.19](https://github.com/dynamic-labs/DynamicAuth/compare/v1.4.18...v1.4.19) (2024-08-23)
+
+
+### Bug Fixes
+
+* allow twitter popup to close ([#6699](https://github.com/dynamic-labs/DynamicAuth/issues/6699)) ([b8f9484](https://github.com/dynamic-labs/DynamicAuth/commit/b8f9484af974efd9be473452ff05de0059e15ac2))
+
+### [1.4.18](https://github.com/dynamic-labs/DynamicAuth/compare/v1.4.17...v1.4.18) (2024-07-31)
+
+
+### Bug Fixes
+
+* sdkHasLoaded flipping back to false with otp ([#6256](https://github.com/dynamic-labs/DynamicAuth/issues/6256)) ([#6451](https://github.com/dynamic-labs/DynamicAuth/issues/6451)) ([b4c86df](https://github.com/dynamic-labs/DynamicAuth/commit/b4c86dff136cc5a866c70610a12ffc75a8ae6d4f)), closes [/github.com/dynamic-labs/dynamic-auth/commit/4fc047f02f6f06d5c129d8de4eed42ee35b4545e#diff-cf5bbeb2e3bd88281d04fe5d30137f9f135011803101c2dec57d5f35cbb47aa0](https://github.com/dynamic-labs//github.com/dynamic-labs/dynamic-auth/commit/4fc047f02f6f06d5c129d8de4eed42ee35b4545e/issues/diff-cf5bbeb2e3bd88281d04fe5d30137f9f135011803101c2dec57d5f35cbb47aa0)
+
 ### [1.4.17](https://github.com/dynamic-labs/DynamicAuth/compare/v1.4.16...v1.4.17) (2024-07-25)
 
 

package/package.cjs

@@ -2,7 +2,7 @@
 
 Object.defineProperty(exports, '__esModule', { value: true });
 
-var version = "1.4.17";
+var version = "1.4.19";
 var dependencies = {
 	"@dynamic-labs/sdk-api": "0.0.387",
 	"@hcaptcha/react-hcaptcha": "1.4.4",

package/package.js

@@ -1,4 +1,4 @@
-var version = "1.4.17";
+var version = "1.4.19";
 var dependencies = {
 	"@dynamic-labs/sdk-api": "0.0.387",
 	"@hcaptcha/react-hcaptcha": "1.4.4",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dynamic-labs/sdk-react-core",
-  "version": "1.4.17",
+  "version": "1.4.19",
   "repository": {
     "type": "git",
     "url": "https://github.com/dynamic-labs/DynamicAuth.git",
@@ -16,16 +16,16 @@
     "react-focus-lock": "2.9.2",
     "react-i18next": "^13.2.0",
     "yup": "0.32.11",
-    "@dynamic-labs/ethereum": "1.4.17",
-    "@dynamic-labs/iconic": "1.4.17",
-    "@dynamic-labs/logger": "1.4.17",
-    "@dynamic-labs/multi-wallet": "1.4.17",
-    "@dynamic-labs/rpc-providers": "1.4.17",
-    "@dynamic-labs/types": "1.4.17",
-    "@dynamic-labs/utils": "1.4.17",
-    "@dynamic-labs/viem-utils": "1.4.17",
-    "@dynamic-labs/wallet-book": "1.4.17",
-    "@dynamic-labs/wallet-connector-core": "1.4.17",
+    "@dynamic-labs/ethereum": "1.4.19",
+    "@dynamic-labs/iconic": "1.4.19",
+    "@dynamic-labs/logger": "1.4.19",
+    "@dynamic-labs/multi-wallet": "1.4.19",
+    "@dynamic-labs/rpc-providers": "1.4.19",
+    "@dynamic-labs/types": "1.4.19",
+    "@dynamic-labs/utils": "1.4.19",
+    "@dynamic-labs/viem-utils": "1.4.19",
+    "@dynamic-labs/wallet-book": "1.4.19",
+    "@dynamic-labs/wallet-connector-core": "1.4.19",
     "eventemitter3": "5.0.1"
   },
   "devDependencies": {

package/src/lib/context/DynamicContext/DynamicContext.cjs

@@ -267,7 +267,7 @@ const InnerDynamicContextProvider = ({ children, theme, settings, locale: locale
         setIsFullyConnected(!(bridgeChainsToConnect === null || bridgeChainsToConnect === void 0 ? void 0 : bridgeChainsToConnect.length) && Boolean(primaryWallet));
     }, [bridgeChainsToConnect, primaryWallet]);
     const walletConnectors = linkedOrConnectedWallets.map((wallet) => wallet.connector);
-    const { sdkHasLoaded, setSdkHasLoaded } = useGlobalLoading.useGlobalLoading({
+    const sdkHasLoaded = useGlobalLoading.useGlobalLoading({
         authMode,
         connectedInfo: connectedWalletsInfo[0],
         connectedWallets,
@@ -309,7 +309,6 @@ const InnerDynamicContextProvider = ({ children, theme, settings, locale: locale
         setLegacyIsVerifying,
         setMultiWalletWidgetState,
         setOnboardingOnlyJwt,
-        setSdkHasLoaded,
         setSelectedWalletConnectorKey,
         setShowAuthFlow,
         setShowBridgeWidget,
@@ -525,7 +524,6 @@ const InnerDynamicContextProvider = ({ children, theme, settings, locale: locale
             setPrimaryWallet,
             setPrimaryWalletId,
             setQrcodeUri,
-            setSdkHasLoaded,
             setSelectedWalletConnectorKey,
             setSelectedWalletWithAction,
             setShowAuthFlow,
@@ -638,7 +636,6 @@ const InnerDynamicContextProvider = ({ children, theme, settings, locale: locale
         setNetwork,
         setPrimaryWallet,
         setPrimaryWalletId,
-        setSdkHasLoaded,
         setSelectedWalletConnectorKey,
         setShowAuthFlow,
         setShowLinkNewWalletModal,

package/src/lib/context/DynamicContext/DynamicContext.js

@@ -263,7 +263,7 @@ const InnerDynamicContextProvider = ({ children, theme, settings, locale, }) =>
         setIsFullyConnected(!(bridgeChainsToConnect === null || bridgeChainsToConnect === void 0 ? void 0 : bridgeChainsToConnect.length) && Boolean(primaryWallet));
     }, [bridgeChainsToConnect, primaryWallet]);
     const walletConnectors = linkedOrConnectedWallets.map((wallet) => wallet.connector);
-    const { sdkHasLoaded, setSdkHasLoaded } = useGlobalLoading({
+    const sdkHasLoaded = useGlobalLoading({
         authMode,
         connectedInfo: connectedWalletsInfo[0],
         connectedWallets,
@@ -305,7 +305,6 @@ const InnerDynamicContextProvider = ({ children, theme, settings, locale, }) =>
         setLegacyIsVerifying,
         setMultiWalletWidgetState,
         setOnboardingOnlyJwt,
-        setSdkHasLoaded,
         setSelectedWalletConnectorKey,
         setShowAuthFlow,
         setShowBridgeWidget,
@@ -521,7 +520,6 @@ const InnerDynamicContextProvider = ({ children, theme, settings, locale, }) =>
             setPrimaryWallet,
             setPrimaryWalletId,
             setQrcodeUri,
-            setSdkHasLoaded,
             setSelectedWalletConnectorKey,
             setSelectedWalletWithAction,
             setShowAuthFlow,
@@ -634,7 +632,6 @@ const InnerDynamicContextProvider = ({ children, theme, settings, locale, }) =>
         setNetwork,
         setPrimaryWallet,
         setPrimaryWalletId,
-        setSdkHasLoaded,
         setSelectedWalletConnectorKey,
         setShowAuthFlow,
         setShowLinkNewWalletModal,

package/src/lib/context/DynamicContext/hooks/useHandleLogout/types.d.ts

@@ -6,7 +6,6 @@ import { ChainsToConnect } from '../../types';
 import { SetShowAuthFlow } from '../useShowAuthFlow';
 export type UseHandleLogoutParams = {
     removeUser: () => void;
-    setSdkHasLoaded: Dispatch<SetStateAction<boolean>>;
     clearAllWalletConnectSessions: () => void;
     bridgeChains: ChainsToConnect[] | undefined;
     setSelectedWalletConnectorKey: Dispatch<SetStateAction<string | null>>;

package/src/lib/context/DynamicContext/hooks/useHandleLogout/useHandleLogout.cjs

@@ -78,7 +78,6 @@ const useHandleLogout = (params) => {
     paramsRef.current = params;
     const logout = React.useCallback(() => _tslib.__awaiter(void 0, void 0, void 0, function* () {
         const p = paramsRef.current;
-        p.setSdkHasLoaded(false);
         p.setSelectedWalletConnectorKey(null);
         p.clearAllWalletConnectSessions();
         p.setShowDynamicUserProfile(false);

package/src/lib/context/DynamicContext/hooks/useHandleLogout/useHandleLogout.js

@@ -74,7 +74,6 @@ const useHandleLogout = (params) => {
     paramsRef.current = params;
     const logout = useCallback(() => __awaiter(void 0, void 0, void 0, function* () {
         const p = paramsRef.current;
-        p.setSdkHasLoaded(false);
         p.setSelectedWalletConnectorKey(null);
         p.clearAllWalletConnectSessions();
         p.setShowDynamicUserProfile(false);

package/src/lib/shared/utils/hooks/useGlobalLoading/useGlobalLoading.cjs

@@ -6,37 +6,30 @@ var React = require('react');
 var index = require('../../functions/lastAuthenticatedAccount/index.cjs');
 
 const useGlobalLoading = ({ authMode, connectedInfo, connectedWallets, projectSettings, primaryWallet, user, walletBook, }) => {
-    var _a;
-    const [sdkHasLoaded, setSdkHasLoaded] = React.useState(false);
-    const primaryWalletIsLoading = user &&
-        ((_a = index.lastAuthenticatedAccount(user)) === null || _a === void 0 ? void 0 : _a.format) === 'blockchain' &&
+    var _a, _b;
+    // Once we declare the SDK as loaded, we do not ever want to flip back to false
+    const hasLoaded = React.useRef(false);
+    const finishLoading = () => {
+        if (hasLoaded.current)
+            return;
+        hasLoaded.current = true;
+    };
+    if (hasLoaded.current)
+        return true;
+    const hasWalletBookLoaded = Boolean(Object.keys((_a = walletBook === null || walletBook === void 0 ? void 0 : walletBook.wallets) !== null && _a !== void 0 ? _a : {}).length);
+    const isPrimaryWalletLoading = user &&
+        ((_b = index.lastAuthenticatedAccount(user)) === null || _b === void 0 ? void 0 : _b.format) === 'blockchain' &&
         !primaryWallet;
-    const connectOnlyUserIsLoading = authMode === 'connect-only' &&
+    const isConnectOnlyWalletsLoading = authMode === 'connect-only' &&
         connectedInfo &&
         connectedWallets.length === 0;
-    React.useEffect(() => {
-        var _a;
-        if (!Object.keys((_a = walletBook === null || walletBook === void 0 ? void 0 : walletBook.wallets) !== null && _a !== void 0 ? _a : {}).length) {
-            return;
-        }
-        if (projectSettings) {
-            if (primaryWalletIsLoading || connectOnlyUserIsLoading) {
-                return;
-            }
-            setSdkHasLoaded(true);
-        }
-    }, [
-        projectSettings,
-        primaryWallet,
-        user,
-        primaryWalletIsLoading,
-        connectOnlyUserIsLoading,
-        walletBook,
-    ]);
-    return {
-        sdkHasLoaded,
-        setSdkHasLoaded,
-    };
+    if (hasWalletBookLoaded &&
+        Boolean(projectSettings) &&
+        !isPrimaryWalletLoading &&
+        !isConnectOnlyWalletsLoading) {
+        finishLoading();
+    }
+    return hasLoaded.current;
 };
 
 exports.useGlobalLoading = useGlobalLoading;

package/src/lib/shared/utils/hooks/useGlobalLoading/useGlobalLoading.d.ts

@@ -1,19 +1,14 @@
-/// <reference types="react" />
-import { ProjectSettings } from '@dynamic-labs/sdk-api';
 import { WalletBookSchema } from '@dynamic-labs/wallet-book';
-import { UserProfile, Wallet } from '../../../types';
 import { ConnectedWalletConnectorType } from '../../../../utils/hooks/useConnectWallet/useConnectWallet';
+import { UserProfile, Wallet } from '../../../types';
 type Props = {
     authMode: string;
     connectedInfo: ConnectedWalletConnectorType | null;
     connectedWallets: Wallet[];
     primaryWallet: Wallet | null;
-    projectSettings: ProjectSettings | undefined;
+    projectSettings: any;
     user: UserProfile | undefined;
     walletBook: WalletBookSchema;
 };
-export declare const useGlobalLoading: ({ authMode, connectedInfo, connectedWallets, projectSettings, primaryWallet, user, walletBook, }: Props) => {
-    sdkHasLoaded: boolean;
-    setSdkHasLoaded: import("react").Dispatch<import("react").SetStateAction<boolean>>;
-};
+export declare const useGlobalLoading: ({ authMode, connectedInfo, connectedWallets, projectSettings, primaryWallet, user, walletBook, }: Props) => boolean;
 export {};

package/src/lib/shared/utils/hooks/useGlobalLoading/useGlobalLoading.js

@@ -1,38 +1,31 @@
-import { useState, useEffect } from 'react';
+import { useRef } from 'react';
 import { lastAuthenticatedAccount } from '../../functions/lastAuthenticatedAccount/index.js';
 
 const useGlobalLoading = ({ authMode, connectedInfo, connectedWallets, projectSettings, primaryWallet, user, walletBook, }) => {
-    var _a;
-    const [sdkHasLoaded, setSdkHasLoaded] = useState(false);
-    const primaryWalletIsLoading = user &&
-        ((_a = lastAuthenticatedAccount(user)) === null || _a === void 0 ? void 0 : _a.format) === 'blockchain' &&
+    var _a, _b;
+    // Once we declare the SDK as loaded, we do not ever want to flip back to false
+    const hasLoaded = useRef(false);
+    const finishLoading = () => {
+        if (hasLoaded.current)
+            return;
+        hasLoaded.current = true;
+    };
+    if (hasLoaded.current)
+        return true;
+    const hasWalletBookLoaded = Boolean(Object.keys((_a = walletBook === null || walletBook === void 0 ? void 0 : walletBook.wallets) !== null && _a !== void 0 ? _a : {}).length);
+    const isPrimaryWalletLoading = user &&
+        ((_b = lastAuthenticatedAccount(user)) === null || _b === void 0 ? void 0 : _b.format) === 'blockchain' &&
         !primaryWallet;
-    const connectOnlyUserIsLoading = authMode === 'connect-only' &&
+    const isConnectOnlyWalletsLoading = authMode === 'connect-only' &&
         connectedInfo &&
         connectedWallets.length === 0;
-    useEffect(() => {
-        var _a;
-        if (!Object.keys((_a = walletBook === null || walletBook === void 0 ? void 0 : walletBook.wallets) !== null && _a !== void 0 ? _a : {}).length) {
-            return;
-        }
-        if (projectSettings) {
-            if (primaryWalletIsLoading || connectOnlyUserIsLoading) {
-                return;
-            }
-            setSdkHasLoaded(true);
-        }
-    }, [
-        projectSettings,
-        primaryWallet,
-        user,
-        primaryWalletIsLoading,
-        connectOnlyUserIsLoading,
-        walletBook,
-    ]);
-    return {
-        sdkHasLoaded,
-        setSdkHasLoaded,
-    };
+    if (hasWalletBookLoaded &&
+        Boolean(projectSettings) &&
+        !isPrimaryWalletLoading &&
+        !isConnectOnlyWalletsLoading) {
+        finishLoading();
+    }
+    return hasLoaded.current;
 };
 
 export { useGlobalLoading };

package/src/lib/utils/hooks/useSocialAuth/useSocialAuth.cjs

@@ -98,7 +98,7 @@ const socialProviders = [
 // pkce params are only required for twitter and break if added for other providers, like google
 // so we should only add provider that support pkce to this list
 const providersRequiringPkce = ['twitter', 'github'];
-const providersWithoutWindowOpenerRefence = ['twitter'];
+const providersWithoutWindowOpenerRefence = [];
 // Hook to handle common social auth logic (for social linking and social sign in)
 const useSocialAuth = ({ sessionTimeout, authMode, onSettled, onError, }) => {
     var _a;

package/src/lib/utils/hooks/useSocialAuth/useSocialAuth.js

@@ -94,7 +94,7 @@ const socialProviders = [
 // pkce params are only required for twitter and break if added for other providers, like google
 // so we should only add provider that support pkce to this list
 const providersRequiringPkce = ['twitter', 'github'];
-const providersWithoutWindowOpenerRefence = ['twitter'];
+const providersWithoutWindowOpenerRefence = [];
 // Hook to handle common social auth logic (for social linking and social sign in)
 const useSocialAuth = ({ sessionTimeout, authMode, onSettled, onError, }) => {
     var _a;