saafe-redirection-flow 2.2.0 → 2.2.2

package/CHANGELOG.md

@@ -1,3 +1,18 @@
+## [2.2.2](https://gitlab.com/Networth360/saafe-redirection/compare/v2.2.1...v2.2.2) (2025-07-24)
+
+
+### Bug Fixes
+
+* event propogation with account selection allowed ([af8543d](https://gitlab.com/Networth360/saafe-redirection/commit/af8543d5ed871be76a2dc2ab7de5c062b2be9e21))
+
+## [2.2.1](https://gitlab.com/Networth360/saafe-redirection/compare/v2.2.0...v2.2.1) (2025-07-24)
+
+
+### Bug Fixes
+
+* linked accounts under manual discovery ([e2c0eb1](https://gitlab.com/Networth360/saafe-redirection/commit/e2c0eb18f291b2ddc8e5c6569b99695337e3d560))
+* linked accounts under manual discovery flow ([5eeaa12](https://gitlab.com/Networth360/saafe-redirection/commit/5eeaa127370b6d9cb3c413744e62b9c91ea4579a))
+
 # [2.2.0](https://gitlab.com/Networth360/saafe-redirection/compare/v2.1.0...v2.2.0) (2025-07-16)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "saafe-redirection-flow",
-  "version": "2.2.0",
+  "version": "2.2.2",
   "type": "module",
   "scripts": {
     "dev": "vite",

package/src/components/LinkedAccountTypeAccordion.tsx

@@ -23,12 +23,16 @@ const LinkedAccountTypeAccordion = ({ category, subType, shouldShowDiscoveryMore
   const { accountForConsent, setAccountForConsent } = useFipStore();
   const [open, setOpen] = useState(false);
 
-  const toggleBankSelection = (bankId: string) => {
-    setAccountForConsent(accountForConsent.includes(bankId)
-      ? accountForConsent.filter(id => id !== bankId)
-      : [...accountForConsent, bankId]
-    );
-  };
+  const toggleBankSelection = (bankId: string, event : React.MouseEvent ) => {
+     if (event) {
+    event.stopPropagation();
+  }
+  
+  setAccountForConsent(accountForConsent.includes(bankId)
+    ? accountForConsent.filter(id => id !== bankId)
+    : [...accountForConsent, bankId]
+  );
+};
 
   // flattened array for achieving select all functionality
   const flattenSubType = Object.values(subType || {}).flat();
@@ -57,7 +61,7 @@ const LinkedAccountTypeAccordion = ({ category, subType, shouldShowDiscoveryMore
           <div>
             <div className='flex items-center text-stone-400 justify-between gap-2 cursor-pointer'>
               <div className='flex flex-col justify-center relative'>
-                <p className="text-md text-stone-500 dark:text-gray-200 font-bold">
+                <p className="text-md text-stone-500 dark:text-gray-200 font-semibold">
                   {category === "BANK" ? "Bank Account" : `${category}`}
                 </p>
                 <p className="text-sm text-stone-700 dark:text-gray-400">
@@ -66,7 +70,7 @@ const LinkedAccountTypeAccordion = ({ category, subType, shouldShowDiscoveryMore
               </div>
               <div>
                 <ChevronDownIcon
-                  size={20}
+                  size={18}
                   className={`h-5 w-5 transition-transform duration-300 ${
                     open ? "rotate-180" : "rotate-0"
                   }`}
@@ -75,18 +79,19 @@ const LinkedAccountTypeAccordion = ({ category, subType, shouldShowDiscoveryMore
             </div>
           </div>
         </CollapsibleTrigger>
-        <CollapsibleContent open={open}>
-          <div className='flex flex-col gap-2 mt-4'>
-            <div className="flex justify-between items-center">
-              <SectionTitle title={"Linked accounts"} rightSection={
-                open ? (
+       <CollapsibleContent open={open}>
+        <div className='flex flex-col gap-2 mt-4'>
+          <div className="flex justify-between items-center">
+            <SectionTitle   className='text-sm md:text-sm font-none' title={"Linked accounts"} rightSection={
+              open ? (
                 <div className="flex justify-end">
                   <AnimatedButton
                     variant={'text'}
                     size={'text'}
                     className='text-primary'
                     onClick={(e) => {
-                      e.stopPropagation();
+                      // e.stopPropagation();
+                      e.preventDefault();
                       handleSelectAll();
                     }}
                   >
@@ -96,56 +101,62 @@ const LinkedAccountTypeAccordion = ({ category, subType, shouldShowDiscoveryMore
                   </AnimatedButton>
                 </div>
               ) : null
-              }/>
-            </div>
-            {Object.entries(subType || {}).map(([fiType, accounts], index) => {
-              return (
-                <div key={fiType}>
-                  <p className="text-stone-400">{fiType}</p>
-                  {accounts.map(account => {
-                    return (
-                      <div className='mt-2 flex flex-col gap-2' key={account.linkRefNumber}>
-                        <OuterCard
-                          selected={accountForConsent.includes(account.linkRefNumber)}
-                          onSelect={() => toggleBankSelection(account.linkRefNumber)}
-                        >
-                          <BankCard
-                            bankName={account.bankName}
-                            image={account.logoUrl}
-                            subText={`${account.accountType} | **${formatAccountNumber(account.maskedAccNumber)}`}
-                            rightSection={
-                              <Checkbox
-                                id={`checkbox-${account.linkRefNumber}`}
-                                checked={accountForConsent.includes(account.linkRefNumber)}
-                                onCheckedChange={() => toggleBankSelection(account.linkRefNumber)}
-                              />
-                            }
-                          />
-                        </OuterCard>
-                      </div>
-                    );
-                  })}
-                </div>
-              );
-            })}
+            }/>
           </div>
-          {shouldShowDiscoveryMore(category || '') && (
-            <Button
-              variant={'outline'}
-              className='bg-primary-foreground text-primary border-primary hover:bg-primary/10 h-[50px] w-full mt-4'
-              onClick={() => handleDiscoveryMore(category || '')}
-              size={'lg'}
-            >
-              <div className='flex items-center gap-2 text-md text-primary font-[600] dark:text-muted-secondary'>
-                <PlusIcon
-                  strokeWidth='2.3'
-                  className='size-5 text-primary dark:text-muted-secondary'
-                />
-                {t('discoverMore')}
+          
+          {Object.entries(subType || {}).map(([fiType, accounts], index) => {
+            return (
+              <div key={fiType}>
+                <p className="text-stone-400 text-sm md:text-sm">{fiType}</p>
+                {accounts.map(account => {
+                  const isSelected = accountForConsent.includes(account.linkRefNumber);
+
+                  return (
+                    <div className='mt-2 flex flex-col gap-2' key={account.linkRefNumber} onClick={() => toggleBankSelection(account.linkRefNumber)}>
+                      <OuterCard
+                        selected={isSelected}
+                        onSelect={() => toggleBankSelection(account.linkRefNumber)}
+                      >
+                        <BankCard
+                          bankName={account.bankName}
+                          image={account.logoUrl}
+                          subText={`${account.accountType} | **${formatAccountNumber(account.maskedAccNumber)}`}
+                          rightSection={
+                            <Checkbox
+                              id={`checkbox-${account.linkRefNumber}`}
+                              checked={isSelected}
+                              onCheckedChange={() => toggleBankSelection(account.linkRefNumber)}
+                              onClick={(e) => e.stopPropagation()}
+                          
+                            />
+                          }
+                        />
+                      </OuterCard>
+                    </div>
+                  );
+                })}
               </div>
-            </Button>
-          )}
-        </CollapsibleContent>
+            );
+          })}
+        </div>
+  
+        {shouldShowDiscoveryMore(category || '') && (
+          <Button
+            variant={'outline'}
+            className='bg-primary-foreground text-primary border-primary hover:bg-primary/10 h-[50px] w-full mt-4'
+            onClick={() => handleDiscoveryMore(category || '')}
+            size={'lg'}
+          >
+            <div className='flex items-center gap-2 text-md text-primary font-[600] dark:text-muted-secondary'>
+              <PlusIcon
+                strokeWidth='2.3'
+                className='size-5 text-primary dark:text-muted-secondary'
+              />
+              {t('discoverMore')}
+            </div>
+          </Button>
+        )}
+</       CollapsibleContent>
       </Collapsible>
     </div>
   );

package/src/components/auth/AuthGuard.tsx

@@ -59,9 +59,10 @@ const AuthGuard = ({ children, requireAuth = true }: AuthGuardProps) => {
     //   }
     // }
 
-    if (decodedInfo.isUserPresent) {
-      return <Navigate to="/link-accounts/old-user" replace />;
-    } else if (decodedInfo?.fiTypesRequiredForConsent.length > 0) {
+    // if (decodedInfo.isUserPresent) {
+    //   return <Navigate to="/link-accounts/old-user" replace />;
+    // } else 
+    if (decodedInfo?.fiTypesRequiredForConsent.length > 0) {
       const fiTypes = extractCategories(decodedInfo?.fiTypesRequiredForConsent);
       setActiveCategory(fiTypes?.[0]);
       return <Navigate to={`/link-accounts/${fiTypes?.[0]?.toLowerCase()}`} replace />;

package/src/components/title/AppBar.tsx

@@ -68,8 +68,11 @@ const AppBar = ({
     if (currentIndex < categories.length - 1) {
       // Move to the next category
       const nextCategory = categories[currentIndex + 1]
-      setActiveCategory(nextCategory)
+      
 
+      // setActiveCategory(nextCategory) //BUG : Changing activeCategory to BANK
+
+      
       // Navigate directly to the next category path for proper step highlighting
       navigate(`/link-accounts/${nextCategory.toLowerCase()}`, {
         state: {

package/src/components/title/SectionTitle.tsx

@@ -14,9 +14,9 @@ const SectionTitle = ({
   return (
     <div className='w-full flex flex-col'>
       <div
-        className={`flex items-center gap-4 w-full justify-between ${className}`}
+        className={`flex items-center gap-4 w-full justify-between text-sm md:text-lg ${className}`}
       >
-        <p className='text-sm md:text-lg font-[600] text-consent-primary dark:text-muted-foreground'>
+        <p className='font-[600] text-consent-primary dark:text-muted-foreground'>
           {title}
         </p>
         <div>{rightSection ? rightSection : null}</div>

package/src/hooks/use-fip-query.ts

@@ -6,7 +6,7 @@ import { useRedirectStore } from "@/store/redirect.store";
 /**
  * Helper function to extract categories from API response
  */
-const extractCategoriesFromResponse = (data: FipResponseData): string[] => {
+export const extractCategoriesFromResponse = (data: FipResponseData): string[] => {
   // If response is not an object, return empty array
   if (!data || typeof data !== "object") return [];