Merge pull request #1824 from oasisprotocol/lw/ledger-disabled-paratimes

Show a link to disabled ParaTimes page on Ledger accounts
oasisprotocol · Jan 11, 2024 · c933e1d · c933e1d
2 parents 31509c8 + 7ec365a
commit c933e1d
Show file tree

Hide file tree

Showing 16 changed files with 722 additions and 63 deletions.
diff --git a/.changelog/1824.feature.md b/.changelog/1824.feature.md
@@ -0,0 +1 @@
+Show a link to disabled ParaTimes page on Ledger accounts
diff --git a/src/app/components/MobileFooterNavigation/__tests__/index.test.tsx b/src/app/components/MobileFooterNavigation/__tests__/index.test.tsx
@@ -24,7 +24,6 @@ const renderComponent = (store: any, { walletHasAccounts, isMobile }: MobileFoot
 describe('<MobileFooterNavigation />', () => {
   let store: ReturnType<typeof configureAppStore>
   const mockUseParaTimesNavigationResult = {
-    canAccessParaTimesRoute: false,
     getParaTimesRoutePath: (address: string) => address,
     paraTimesRouteLabel: 'MockParaTimesLabel',
   } as ParaTimesNavigationHook
@@ -47,7 +46,7 @@ describe('<MobileFooterNavigation />', () => {
     renderComponent(store, { walletHasAccounts: true, isMobile: true })
 
     expect(screen.getByTestId('mobile-footer-navigation')).toBeInTheDocument()
-    expect(screen.queryByText('MockParaTimesLabel')).not.toBeInTheDocument()
+    expect(screen.queryByText('MockParaTimesLabel')).toBeInTheDocument()
   })
 
   it('should not render component for non mobile', () => {
@@ -61,15 +60,4 @@ describe('<MobileFooterNavigation />', () => {
 
     expect(screen.queryByTestId('mobile-footer-navigation')).not.toBeInTheDocument()
   })
-
-  it('should render paraTime link', () => {
-    jest.mocked(useParaTimesNavigation).mockReturnValue({
-      ...mockUseParaTimesNavigationResult,
-      canAccessParaTimesRoute: true,
-    })
-    renderComponent(store, { walletHasAccounts: true, isMobile: true })
-
-    expect(screen.getByText('MockParaTimesLabel')).toBeInTheDocument()
-    expect(screen.getByLabelText('Inherit')).toBeInTheDocument()
-  })
 })
diff --git a/src/app/components/MobileFooterNavigation/index.tsx b/src/app/components/MobileFooterNavigation/index.tsx
@@ -40,7 +40,7 @@ export interface MobileFooterNavigationProps {
 export const MobileFooterNavigation = ({ walletHasAccounts, isMobile }: MobileFooterNavigationProps) => {
   const { t } = useTranslation()
   const address = useSelector(selectAddress)
-  const { canAccessParaTimesRoute, getParaTimesRoutePath, paraTimesRouteLabel } = useParaTimesNavigation()
+  const { getParaTimesRoutePath, paraTimesRouteLabel } = useParaTimesNavigation()
   const getMenuItems = useMemo(() => {
     const menuItems = [
       {
@@ -53,15 +53,11 @@ export const MobileFooterNavigation = ({ walletHasAccounts, isMobile }: MobileFo
         Icon: LineChart,
         to: `/account/${address}/stake`,
       },
-      ...(canAccessParaTimesRoute
-        ? [
-            {
-              label: paraTimesRouteLabel,
-              Icon: Inherit,
-              to: getParaTimesRoutePath(address!),
-            },
-          ]
-        : []),
+      {
+        label: paraTimesRouteLabel,
+        Icon: Inherit,
+        to: getParaTimesRoutePath(address!),
+      },
       ...(IS_FIAT_ONRAMP_ENABLED
         ? [
             {
@@ -73,7 +69,7 @@ export const MobileFooterNavigation = ({ walletHasAccounts, isMobile }: MobileFo
         : []),
     ]
     return menuItems
-  }, [address, canAccessParaTimesRoute, getParaTimesRoutePath, paraTimesRouteLabel, t])
+  }, [address, getParaTimesRoutePath, paraTimesRouteLabel, t])
 
   if (!isMobile || !walletHasAccounts) {
     return null