fix: lw-9273 handle successful confirmation window (#779)

input-output-hk · Dec 8, 2023 · 548ce78 · 548ce78
1 parent 8d78ac1
commit 548ce78
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/apps/browser-extension-wallet/src/features/dapp/components/DappTransactionSuccess.tsx b/apps/browser-extension-wallet/src/features/dapp/components/DappTransactionSuccess.tsx
@@ -6,10 +6,12 @@ import Success from '../../../assets/icons/success-staking.svg';
 import styles from './Layout.module.scss';
 import { useAnalyticsContext } from '@providers';
 import { TX_CREATION_TYPE_KEY, TxCreationType } from '@providers/AnalyticsProvider/analyticsTracker';
+import { useWalletManager } from '@hooks';
 
 export const DappTransactionSuccess = (): React.ReactElement => {
   const analytics = useAnalyticsContext();
   const { t } = useTranslation();
+  const { clearPassword } = useWalletManager();
 
   const onClose = async () => {
     await analytics?.sendEventToPostHog(PostHogAction.SendAllDoneCloseClick, {
@@ -18,6 +20,10 @@ export const DappTransactionSuccess = (): React.ReactElement => {
     window.close();
   };
 
+  useEffect(() => {
+    clearPassword();
+  }, [clearPassword]);
+
   useEffect(() => {
     analytics?.sendEventToPostHog(PostHogAction.SendAllDoneView, {
       [TX_CREATION_TYPE_KEY]: TxCreationType.External

diff --git a/apps/browser-extension-wallet/src/hooks/useWalletManager.ts b/apps/browser-extension-wallet/src/hooks/useWalletManager.ts
@@ -60,6 +60,7 @@ export interface UseWalletManager {
   saveHardwareWallet: (wallet: Wallet.CardanoWalletByChain, chainName?: Wallet.ChainName) => Promise<void>;
   deleteWallet: (isForgotPasswordFlow?: boolean) => Promise<void>;
   executeWithPassword: <T>(password: string, promiseFn: () => Promise<T>, cleanPassword?: boolean) => Promise<T>;
+  clearPassword: () => void;
   switchNetwork: (chainName: Wallet.ChainName) => Promise<void>;
   updateAddresses: (args: {
     addresses: Wallet.KeyManagement.GroupedAddress[];
@@ -144,6 +145,13 @@ export const useWalletManager = (): UseWalletManager => {
     [backgroundService]
   );
 
+  /**
+   * Clears the wallet password
+   */
+  const clearPassword = useCallback(() => {
+    backgroundService.setWalletPassword();
+  }, [backgroundService]);
+
   /**
    * Deletes wallet info in storage, which should be stored encrypted with the wallet password as lock
    */
@@ -481,6 +489,7 @@ export const useWalletManager = (): UseWalletManager => {
     saveHardwareWallet,
     deleteWallet,
     executeWithPassword,
+    clearPassword,
     switchNetwork,
     updateAddresses
   };