fix: search and select profile on transfer

WatchItDev · Feb 2, 2025 · 37abe06 · 37abe06
1 parent df1aa7c
commit 37abe06
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 9 deletions.
diff --git a/src/sections/finance/components/finance-display-profile-info.tsx b/src/sections/finance/components/finance-display-profile-info.tsx
@@ -38,7 +38,7 @@ const FinanceDisplayProfileInfo: FC<FinanceDisplayNameProps> = ({initialList, ca
       {
         mode === 'profile' ?
           (<Box component="span" sx={{ flexGrow: 1, typography: 'subtitle1' }}>
-            {selectedProfile?.metadata?.displayName ?? 'No profile selected'}
+            {selectedProfile?.metadata?.displayName ?? selectedProfile?.handle?.localName ?? 'No profile selected'}
           </Box>) : null
       }
       {

diff --git a/src/sections/finance/components/finance-quick-transfer-modal.tsx b/src/sections/finance/components/finance-quick-transfer-modal.tsx
@@ -154,7 +154,7 @@ function FinanceQuickTransferModal({
 
       notifySuccess(SUCCESS.TRANSFER_CREATED_SUCCESSFULLY, {
         destination: isSame
-          ? contactInfo?.metadata?.displayName
+          ? contactInfo?.metadata?.displayName ?? contactInfo?.handle?.localName
           : truncateAddress(address ?? ''),
       });
     } catch (err: any) {

diff --git a/src/sections/finance/components/finance-quick-transfer.tsx b/src/sections/finance/components/finance-quick-transfer.tsx
@@ -103,6 +103,7 @@ export default function FinanceQuickTransfer({
 
       // 3) Move the carousel right now
       carousel.setCurrentIndex(finalIndex);
+      setCurrentIndex(finalIndex);
 
       // 4) Update the wallet address & Redux
       setWalletAddress(profile.ownedBy?.address ?? '');
@@ -254,9 +255,6 @@ export default function FinanceQuickTransfer({
     }
   }, [showRainbow, currentIndex, carousel]);
 
-  // We pick the contactInfo to pass to the modal. If currentIndex is -1, there's no matched profile
-  const contactInfoToPass = currentIndex === -1 ? undefined : getContactInfo;
-
   // Render the carousel of profiles
   const renderCarousel = (
     <Box sx={{ position: 'relative', mb: 3 }}>
@@ -374,7 +372,6 @@ export default function FinanceQuickTransfer({
 
   const Wrapper = showRainbow ? NeonPaper : Box;
 
-  console.log(list);
   return (
     <>
       <Wrapper
@@ -400,9 +397,9 @@ export default function FinanceQuickTransfer({
 
           {/* Content */}
           <Stack sx={{ p: 3 }}>
-            <FinanceDisplayProfileInfo mode={'profile'} initialList={initialList} carousel={carousel} />
+            <FinanceDisplayProfileInfo mode={'profile'} initialList={list} carousel={carousel} />
             {list?.length > 0 ? renderCarousel : <FinanceNoFollowingsQuickTransfer />}
-            <FinanceDisplayProfileInfo mode={'wallet'} initialList={initialList} carousel={carousel} />
+            <FinanceDisplayProfileInfo mode={'wallet'} initialList={list} carousel={carousel} />
             {renderInput}
           </Stack>
         </Stack>
@@ -417,7 +414,7 @@ export default function FinanceQuickTransfer({
         address={walletAddress}
         onClose={confirm.onFalse}
         onFinish={handleTransferFinish}
-        contactInfo={contactInfoToPass} // If currentIndex is -1, this is undefined
+        contactInfo={getContactInfo}
         onChange={handleChangeInput}
       />
     </>

diff --git a/src/utils/wallet.ts b/src/utils/wallet.ts
@@ -2,6 +2,8 @@
 // ----------------------------------------------------------------------
 
 export const truncateAddress = (text: string, startChars: number = 6, endChars: number = 6) => {
+  if (!text) return '';
+
   if (text.length <= startChars + endChars) {
     return text;
   }