From aeb49279fde5fdf290b5662b6a2a99b322516d81 Mon Sep 17 00:00:00 2001
From: Pritish Budhiraja <77892330+PritishBudhiraja@users.noreply.github.com>
Date: Tue, 26 Dec 2023 11:44:50 +0530
Subject: [PATCH] fix: Removed Sublevel remote link type & Pageloader CSS
 fixes. (#120)

---
 src/screens/HyperSwitch/Sidebar/Sidebar.res   | 53 +++++--------------
 .../HyperSwitch/Sidebar/SidebarTypes.res      |  4 +-
 .../Utils/HSwitchGlobalSearchBar.res          |  3 +-
 .../HyperSwitch/Utils/PageLoaderWrapper.res   |  3 +-
 4 files changed, 16 insertions(+), 47 deletions(-)

diff --git a/src/screens/HyperSwitch/Sidebar/Sidebar.res b/src/screens/HyperSwitch/Sidebar/Sidebar.res
index cf2ae62b5..11b063f99 100644
--- a/src/screens/HyperSwitch/Sidebar/Sidebar.res
+++ b/src/screens/HyperSwitch/Sidebar/Sidebar.res
@@ -216,25 +216,6 @@ module NestedSidebarItem = {
             </Link>
           </UIUtils.RenderIf>
         }
-
-      | SubLevelRemoteLink(tabOption) => {
-          let {name, link, access, ?remoteIcon} = tabOption
-          let (remoteUi, link) = if remoteIcon->Belt.Option.getWithDefault(false) {
-            (<Icon name="external-link-alt" size=14 className="ml-3" />, link)
-          } else {
-            (React.null, `${link}${getSearchParamByLink(link)}`)
-          }
-          <UIUtils.RenderIf condition={access !== NoAccess}>
-            <a
-              ref={nestedSidebarItemRef->ReactDOM.Ref.domRef}
-              href={link}
-              target="_blank"
-              className={`${textColor} relative overflow-hidden flex flex-row items-center cursor-pointer ${paddingClass} ${selectedClass} `}>
-              <SidebarSubOption name isSectionExpanded isSelected isSideBarExpanded />
-              remoteUi
-            </a>
-          </UIUtils.RenderIf>
-        }
       }}
     </UIUtils.RenderIf>
   }
@@ -308,23 +289,18 @@ module NestedSectionItem = {
         </UIUtils.RenderIf>
       </div>
       <UIUtils.RenderIf condition={isElementShown}>
-        {
-          let subSectionsArr =
-            section.links
-            ->Array.mapWithIndex((subLevelItem, index) => {
-              let isSelected = subLevelItem->isSubLevelItemSelected
-              <NestedSidebarItem
-                key={string_of_int(index)}
-                isSelected
-                isSideBarExpanded
-                isSectionExpanded
-                tabInfo=subLevelItem
-              />
-            })
-            ->React.array
-
-          subSectionsArr
-        }
+        {section.links
+        ->Array.mapWithIndex((subLevelItem, index) => {
+          let isSelected = subLevelItem->isSubLevelItemSelected
+          <NestedSidebarItem
+            key={string_of_int(index)}
+            isSelected
+            isSideBarExpanded
+            isSectionExpanded
+            tabInfo=subLevelItem
+          />
+        })
+        ->React.array}
       </UIUtils.RenderIf>
     </div>
   }
@@ -341,9 +317,7 @@ module SidebarNestedSection = {
   ) => {
     let isSubLevelItemSelected = tabInfo => {
       switch tabInfo {
-      | SubLevelRemoteLink(item)
-      | SubLevelLink(item) =>
-        linkSelectionCheck(firstPart, item.link)
+      | SubLevelLink(item) => linkSelectionCheck(firstPart, item.link)
       }
     }
 
@@ -410,7 +384,6 @@ module SidebarNestedSection = {
       acc &&
       switch subLevelItem {
       | SubLevelLink({access}) => access === NoAccess
-      | SubLevelRemoteLink({access}) => access === NoAccess
       }
     })
     <UIUtils.RenderIf condition={!areAllSubLevelsHidden}>
diff --git a/src/screens/HyperSwitch/Sidebar/SidebarTypes.res b/src/screens/HyperSwitch/Sidebar/SidebarTypes.res
index 3b8f54486..7433db462 100644
--- a/src/screens/HyperSwitch/Sidebar/SidebarTypes.res
+++ b/src/screens/HyperSwitch/Sidebar/SidebarTypes.res
@@ -27,9 +27,7 @@ type nestedOption = {
   iconSize?: int,
 }
 
-type subLevelItem =
-  | SubLevelRemoteLink(nestedOption)
-  | SubLevelLink(nestedOption)
+type subLevelItem = SubLevelLink(nestedOption)
 
 type sectionType = {
   name: string,
diff --git a/src/screens/HyperSwitch/Utils/HSwitchGlobalSearchBar.res b/src/screens/HyperSwitch/Utils/HSwitchGlobalSearchBar.res
index 5e0dbd867..2209ed1d8 100644
--- a/src/screens/HyperSwitch/Utils/HSwitchGlobalSearchBar.res
+++ b/src/screens/HyperSwitch/Utils/HSwitchGlobalSearchBar.res
@@ -104,8 +104,7 @@ let make = () => {
             [],
             (insideAcc, item) => {
               switch item {
-              | SubLevelLink(obj)
-              | SubLevelRemoteLink(obj) => {
+              | SubLevelLink(obj) => {
                   if (
                     checkStringStartsWithSubstring(~itemToCheck=sectionObj.name, ~searchText) ||
                     checkStringStartsWithSubstring(~itemToCheck=obj.name, ~searchText)
diff --git a/src/screens/HyperSwitch/Utils/PageLoaderWrapper.res b/src/screens/HyperSwitch/Utils/PageLoaderWrapper.res
index 02cf33e39..ec220919a 100644
--- a/src/screens/HyperSwitch/Utils/PageLoaderWrapper.res
+++ b/src/screens/HyperSwitch/Utils/PageLoaderWrapper.res
@@ -20,9 +20,8 @@ let make = (
     }
   | Error(_err) =>
     <DefaultLandingPage
-      height=sectionHeight
       title="Oops, we hit a little bump on the road!"
-      customStyle={`py-16 !m-0 ${customStyleForDefaultLandingPage}`}
+      customStyle={`py-16 !m-0 ${customStyleForDefaultLandingPage} ${sectionHeight}`}
       overriddingStylesTitle="text-2xl font-semibold"
       buttonText="Refresh"
       overriddingStylesSubtitle="!text-sm text-grey-700 opacity-50 !w-3/4"