Merge pull request #3283 from getAlby/allow-to-close-alert-banner

feat: allow to close wallet update banner

src/app/components/Alert/index.tsx

@@ -1,22 +1,51 @@
+import { PopiconsXLine } from "@popicons/react";
+import { useState } from "react";
 import { classNames } from "~/app/utils";
 
 type Props = {
   type: "warn" | "info";
   children: React.ReactNode;
+  showClose?: boolean;
+  onClose?: () => void; // Optional callback function
 };
 
-export default function Alert({ type, children }: Props) {
+export default function Alert({
+  type,
+  children,
+  showClose = false,
+  onClose,
+}: Props) {
+  const [isVisible, setIsVisible] = useState(true);
+
+  const handleClose = () => {
+    setIsVisible(false);
+    if (onClose) {
+      onClose(); // Invoke the callback function if provided
+    }
+  };
+
+  if (!isVisible) return null;
+
   return (
     <div
       className={classNames(
-        "border rounded-md p-3",
-        type == "warn" &&
-          "text-orange-700 dark:text-orange-300 bg-orange-50 dark:bg-orange-900  border-orange-100 dark:border-orange-900",
-        type == "info" &&
+        "border rounded-md p-3 flex justify-between relative",
+        type === "warn" &&
+          "text-orange-700 dark:text-orange-300 bg-orange-50 dark:bg-orange-900 border-orange-100 dark:border-orange-900",
+        type === "info" &&
           "text-blue-700 dark:text-blue-300 bg-blue-50 dark:bg-blue-900 border-blue-100 dark:border-blue-900"
       )}
     >
-      {children}
+      {showClose && (
+        <button
+          onClick={handleClose}
+          className="absolute right-2 top-2 text-gray-600 dark:text-neutral-400 hover:text-gray-700 dark:hover:text-neutral-300"
+          aria-label="Close alert"
+        >
+          <PopiconsXLine className="w-5 h-5" />
+        </button>
+      )}
+      <div className="pr-8">{children}</div>
     </div>
   );
 }

src/app/screens/Home/DefaultView/index.tsx

@@ -55,6 +55,8 @@ const DefaultView: FC<Props> = (props) => {
   const [isBlockedUrl, setIsBlockedUrl] = useState<boolean>(false);
   const [currentAccount, setCurrentAccount] = useState<GetAccountRes>();
   const [nostrPublicKey, setNostrPublicKey] = useState("");
+  const [seenSharedNodeBanner, setSeenSharedNodeBanner] =
+    useState<boolean>(false);
 
   const { transactions, isLoadingTransactions, loadTransactions } =
     useTransactions();
@@ -85,6 +87,8 @@ const DefaultView: FC<Props> = (props) => {
         const userAccount = await api.getAccount();
         const nostrPrivateKey = await api.nostr.getPrivateKey(userAccount.id);
 
+        setSeenSharedNodeBanner(userAccount.seenSharedNodeBanner);
+
         setNostrPublicKey(
           nostrPrivateKey ? await nostr.derivePublicKey(nostrPrivateKey) : ""
         );
@@ -176,8 +180,16 @@ const DefaultView: FC<Props> = (props) => {
           </Alert>
         )}
 
-        {account?.sharedNode && (
-          <Alert type="info">
+        {account?.sharedNode && !seenSharedNodeBanner && (
+          <Alert
+            type="info"
+            showClose
+            onClose={async () =>
+              await api.editAccount(account.id, {
+                seenSharedNodeBanner: true,
+              })
+            }
+          >
             <div className="flex items-center gap-2">
               <div className="shrink-0">
                 <PopiconsCircleExclamationLine className="w-5 h-5" />

src/common/lib/api.ts

@@ -75,6 +75,7 @@ export interface GetAccountRes extends Pick<Account, "id" | "name"> {
   hasMnemonic: boolean;
   isMnemonicBackupDone: boolean;
   hasImportedNostrKey: boolean;
+  seenSharedNodeBanner: boolean;
   bitcoinNetwork: BitcoinNetworkType;
   useMnemonicForLnurlAuth: boolean;
 }

src/extension/background-script/actions/accounts/__tests__/get.test.ts

@@ -63,6 +63,7 @@ describe("account info", () => {
       nostrEnabled: false,
       liquidEnabled: false,
       hasMnemonic: false,
+      seenSharedNodeBanner: false,
       hasImportedNostrKey: true,
       bitcoinNetwork: "bitcoin",
       useMnemonicForLnurlAuth: false,
@@ -91,6 +92,7 @@ describe("account info", () => {
       nostrEnabled: true,
       liquidEnabled: true,
       hasMnemonic: true,
+      seenSharedNodeBanner: false,
       hasImportedNostrKey: true,
       bitcoinNetwork: "regtest",
       useMnemonicForLnurlAuth: true,

src/extension/background-script/actions/accounts/edit.ts

@@ -27,6 +27,11 @@ const edit = async (message: MessageAccountEdit) => {
         message.args.isMnemonicBackupDone;
     }
 
+    if (message.args.seenSharedNodeBanner !== undefined) {
+      accounts[accountId].seenSharedNodeBanner =
+        message.args.seenSharedNodeBanner;
+    }
+
     state.setState({ accounts });
     // make sure we immediately persist the updated accounts
     await state.getState().saveToStorage();

src/extension/background-script/actions/accounts/get.ts

@@ -30,6 +30,7 @@ const get = async (message: MessageAccountGet) => {
     // Note: undefined (default for new accounts) it is also considered imported
     hasImportedNostrKey: account.hasImportedNostrKey !== false,
     bitcoinNetwork: account.bitcoinNetwork || "bitcoin",
+    seenSharedNodeBanner: account.seenSharedNodeBanner || false,
     useMnemonicForLnurlAuth: account.useMnemonicForLnurlAuth || false,
   };
 

src/types.ts

@@ -27,6 +27,7 @@ export interface Account {
   nostrPrivateKey?: string | null;
   mnemonic?: string | null;
   hasImportedNostrKey?: boolean;
+  seenSharedNodeBanner?: boolean;
   bitcoinNetwork?: BitcoinNetworkType;
   isMnemonicBackupDone?: boolean;
   useMnemonicForLnurlAuth?: boolean;
@@ -269,6 +270,7 @@ export interface MessageAccountEdit extends MessageDefault {
     bitcoinNetwork?: BitcoinNetworkType;
     useMnemonicForLnurlAuth?: boolean;
     isMnemonicBackupDone?: boolean;
+    seenSharedNodeBanner?: boolean;
   };
   action: "editAccount";
 }