Merge branch 'test-coverage-dbot' into shafin/BOT-633/chore--test-cas…

…e-recent-footer

.circleci/config.yml

@@ -223,15 +223,17 @@ jobs:
           name: "Check TypeScript for @deriv/stores"
           command: npx tsc --project packages/stores/tsconfig.json -noEmit
       - run:
-          name: "Check TypeScript and linting for @deriv/wallets"
-          command: |
-            npx tsc --project packages/wallets/tsconfig.json -noEmit
-            npx eslint --fix --config packages/wallets/.eslintrc.js packages/wallets
-            npx stylelint packages/wallets/**/*.scss
-
+          name: "Check TypeScript for @deriv/wallets"
+          command: npx tsc --project packages/wallets/tsconfig.json -noEmit
       # - run:
       #     name: "Check TypeScript for @deriv/cashier"
       #     command: npx tsc --project packages/cashier/tsconfig.json -noEmit
+      - run:
+          name: "Check ESLint for @deriv/wallets"
+          command: npx eslint --fix --config packages/wallets/.eslintrc.js packages/wallets
+      - run:
+          name: "Check Stylelint for @deriv/wallets"
+          command: npx stylelint packages/wallets/**/*.scss
       - run:
           name: "Check tests for @deriv/hooks"
           command: bash ./scripts/check-tests.sh packages/hooks/src

.github/CODEOWNERS

@@ -155,3 +155,15 @@
 /packages/trader/**/*                                           @matin-deriv @maryia-deriv
 
 
+# ==============================================================
+#  deriv-app/wallets
+# ==============================================================
+
+/packages/wallets/**/*                                          @adrienne-deriv @thisyahlen-deriv @farhan-nurzi-deriv
+
+
+# ==============================================================
+#  deriv-app/api
+# ==============================================================
+
+/packages/api/**/*                                              @adrienne-deriv @thisyahlen-deriv @farhan-nurzi-deriv

packages/account/src/Components/Routes/helpers.ts

@@ -2,7 +2,7 @@ import { match, matchPath, RouteProps } from 'react-router';
 import { routes } from '@deriv/shared';
 import { TRouteConfig } from 'Types';
 
-export const normalizePath = (path: string) => (/^\//.test(path) ? path : `/${path || ''}`); // Default to '/'
+export const normalizePath = (path: string) => (path.startsWith('/') ? path : `/${path || ''}`); // Default to '/'
 
 export const findRouteByPath = (path: string, routes_config?: TRouteConfig[]): RouteProps | undefined => {
     let result: RouteProps | undefined;
@@ -31,7 +31,7 @@ export const findRouteByPath = (path: string, routes_config?: TRouteConfig[]): R
 };
 
 export const isRouteVisible = (route?: { is_authenticated: boolean }, is_logged_in?: boolean) =>
-    !(route && route.is_authenticated && !is_logged_in);
+    !(route?.is_authenticated && !is_logged_in);
 
 export const getPath = (route_path: string, params: { [key: string]: string } = {}) =>
     Object.keys(params).reduce((p, name) => p.replace(`:${name}`, params[name]), route_path);

packages/account/src/Components/Routes/route-with-sub-routes.tsx

@@ -51,7 +51,7 @@ const RouteWithSubRoutes = (route: TRouteWithSubRoutesProps) => {
             );
         }
 
-        const title = route.getTitle?.() || '';
+        const title = route.getTitle?.() ?? '';
         document.title = `${title} | ${default_title}`;
 
         alternateLinkTagChange();

packages/account/src/Components/account-limits/account-limits-table-cell.tsx

@@ -40,7 +40,7 @@ const AccountLimitsTableCell = ({
                         {children}
                     </Text>
                 )}
-                {renderExtraInfo && renderExtraInfo()}
+                {renderExtraInfo?.()}
             </td>
         </React.Fragment>
     );

packages/account/src/Components/account-limits/account-limits-table-header.tsx

@@ -33,7 +33,7 @@ const AccountLimitsTableHeader = ({
                     {children}
                 </Text>
             )}
-            {renderExtraInfo && renderExtraInfo()}
+            {renderExtraInfo?.()}
         </th>
     );
 };