Merge pull request #3957 from IntersectMBO/test

bosko-m · web-flow · commit 1d3ecb416c28 · 2025-07-24T20:55:34.000+02:00
v2.0.31.1
diff --git a/govtool/frontend/src/App.tsx b/govtool/frontend/src/App.tsx
@@ -63,28 +63,47 @@ export default () => {
   }, []);
 
   const checkTheWalletIsActive = useCallback(() => {
-    const hrefCondition =
-      window.location.pathname === PATHS.home ||
-      window.location.pathname === PATHS.governanceActions ||
-      window.location.pathname === PATHS.governanceActionsAction;
+    const isWalletAvailable = () =>
+        window.cardano && walletName && Object.keys(window.cardano).includes(walletName);
 
-    const walletName = getItemFromLocalStorage(`${WALLET_LS_KEY}_name`);
-    if (window.cardano) {
-      const walletExtensions = Object.keys(window.cardano);
-      if (walletName && walletExtensions.includes(walletName)) {
-        enable(walletName);
-        return;
+    const cleanUpWalletData = () => {
+      removeItemFromLocalStorage(`${WALLET_LS_KEY}_name`);
+      removeItemFromLocalStorage(`${WALLET_LS_KEY}_stake_key`);
+    };
+
+    const waitForWalletExtension = async () => {
+      const timeout = 5000;
+      const interval = 100;
+      const startTime = Date.now();
+
+      while (Date.now() - startTime < timeout) {
+        if (isWalletAvailable()) {
+          enable(walletName);
+          return;
+        }
+        // eslint-disable-next-line no-await-in-loop
+        await new Promise((resolve) => {
+          setTimeout(resolve, interval);
+        });
       }
-    }
-    if (
-      (!window.cardano && walletName) ||
-      (walletName && !Object.keys(window.cardano).includes(walletName))
-    ) {
-      if (!hrefCondition) {
+
+      if (!isOnAllowedPage) {
         navigate(PATHS.home);
       }
-      removeItemFromLocalStorage(`${WALLET_LS_KEY}_name`);
-      removeItemFromLocalStorage(`${WALLET_LS_KEY}_stake_key`);
+      cleanUpWalletData();
+    };
+
+    const isOnAllowedPage = [PATHS.home, PATHS.governanceActions, PATHS.governanceActionsAction]
+        .includes(window.location.pathname);
+
+    const walletName = getItemFromLocalStorage(`${WALLET_LS_KEY}_name`);
+
+    if (!walletName) return;
+
+    if (isWalletAvailable()) {
+      enable(walletName);
+    } else {
+      waitForWalletExtension();
     }
   }, []);
 
diff --git a/govtool/frontend/src/components/atoms/types.ts b/govtool/frontend/src/components/atoms/types.ts
@@ -1,4 +1,5 @@
 import { ChangeEvent } from "react";
+import { LinkProps } from "react-router-dom";
 import {
   ButtonProps as MUIButtonProps,
   CheckboxProps as MUICheckboxProps,
@@ -13,6 +14,10 @@ export type ButtonProps = Omit<MUIButtonProps, "size"> & {
   isLoading?: boolean;
   size?: "small" | "medium" | "large" | "extraLarge";
   dataTestId?: string;
+  to?: LinkProps["to"];
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+  state?: any;
+  component?: React.ElementType;
 };
 
 export type LoadingButtonProps = ButtonProps & {
diff --git a/govtool/frontend/src/components/molecules/GovernanceActionCard.tsx b/govtool/frontend/src/components/molecules/GovernanceActionCard.tsx
@@ -1,5 +1,6 @@
 import { FC } from "react";
 import { Box, Skeleton } from "@mui/material";
+import { Link, useLocation } from "react-router-dom";
 
 import { Button } from "@atoms";
 import {
@@ -19,13 +20,7 @@ import { ProposalData } from "@models";
 
 type ActionTypeProps = Omit<
   ProposalData,
-  | "yesVotes"
-  | "noVotes"
-  | "abstainVotes"
   | "id"
-  | "details"
-  | "rationale"
-  | "motivation"
 > & {
   onClick?: () => void;
   inProgress?: boolean;
@@ -47,6 +42,7 @@ export const GovernanceActionCard: FC<ActionTypeProps> = ({
   title,
   isValidating,
   metadataStatus,
+  ...otherProposalData
 }) => {
   const { isMobile, screenWidth } = useScreenDimension();
   const { t } = useTranslation();
@@ -58,6 +54,9 @@ export const GovernanceActionCard: FC<ActionTypeProps> = ({
     bech32Prefix: "gov_action",
   });
 
+  const pathname = useLocation().pathname.replace(/governance_actions.*/g, "governance_actions");
+  const isCategoryView = useLocation().pathname.includes("category");
+
   return (
     <Box
       sx={{
@@ -151,6 +150,26 @@ export const GovernanceActionCard: FC<ActionTypeProps> = ({
         ) : (
           <Button
             onClick={onClick}
+            component={Link}
+            to={`${pathname}/${govActionId}`}
+            state={{
+              proposal: {
+                abstract,
+                type,
+                inProgress,
+                expiryDate,
+                expiryEpochNo,
+                createdDate,
+                createdEpochNo,
+                txHash,
+                index,
+                title,
+                isValidating,
+                metadataStatus,
+                ...otherProposalData,
+              },
+              openedFromCategoryPage: isCategoryView
+            }}
             variant={inProgress ? "outlined" : "contained"}
             size="large"
             sx={{
diff --git a/govtool/frontend/src/components/organisms/DashboardGovernanceActionsVotedOn.tsx b/govtool/frontend/src/components/organisms/DashboardGovernanceActionsVotedOn.tsx
@@ -26,20 +26,39 @@ export const DashboardGovernanceActionsVotedOn = ({
   const { t } = useTranslation();
 
   // TODO: Filtering here is some kind of craziness. It should be done on the backend.
+
   const filteredData = useMemo(() => {
-    if (votes.length && searchPhrase) {
-      return votes
-        .map((entry) => ({
+    if (!votes?.length) return [];
+    if (!searchPhrase) return votes;
+    const lowerSearch = searchPhrase.toLowerCase();
+    return votes
+      .map((entry) => {
+        const filteredActions = entry.actions.filter((action) => {
+          const hash = getFullGovActionId(
+            action.proposal.txHash,
+            action.proposal.index,
+          ).toLowerCase();
+
+          const title = action.proposal.title?.toLowerCase() || "";
+          const motivation = action.proposal.motivation?.toLowerCase() || "";
+          const rationale = action.proposal.rationale?.toLowerCase() || "";
+          const abstract = action.proposal.abstract?.toLowerCase() || "";
+
+          return (
+            hash.includes(lowerSearch) ||
+            title.includes(lowerSearch) ||
+            motivation.includes(lowerSearch) ||
+            rationale.includes(lowerSearch) ||
+            abstract.includes(lowerSearch)
+          );
+        });
+
+        return {
           ...entry,
-          actions: entry.actions.filter((action) =>
-            getFullGovActionId(action.proposal.txHash, action.proposal.index)
-              .toLowerCase()
-              .includes(searchPhrase.toLowerCase()),
-          ),
-        }))
-        .filter((entry) => entry.actions?.length > 0);
-    }
-    return votes;
+          actions: filteredActions,
+        };
+      })
+      .filter((entry) => entry.actions.length > 0);
   }, [votes, searchPhrase, pendingTransaction.vote]);
 
   return areDRepVotesLoading ? (
diff --git a/govtool/frontend/src/components/organisms/GovernanceActionsToVote.tsx b/govtool/frontend/src/components/organisms/GovernanceActionsToVote.tsx
@@ -1,12 +1,10 @@
-import { useNavigate, generatePath } from "react-router-dom";
 import { Box } from "@mui/material";
 
 import { Typography } from "@atoms";
-import { PATHS } from "@consts";
 import { useCardano } from "@context";
 import { useScreenDimension, useTranslation } from "@hooks";
 import { ProposalData } from "@models";
-import { getProposalTypeTitle, getFullGovActionId } from "@utils";
+import { getProposalTypeTitle } from "@utils";
 import { Slider, ValidatedGovernanceActionCard } from "@organisms";
 
 type GovernanceActionsToVoteProps = {
@@ -25,7 +23,6 @@ export const GovernanceActionsToVote = ({
   sorting,
 }: GovernanceActionsToVoteProps) => {
   const { pendingTransaction } = useCardano();
-  const navigate = useNavigate();
   const { isMobile, pagePadding } = useScreenDimension();
   const { t } = useTranslation();
 
@@ -64,29 +61,6 @@ export const GovernanceActionsToVote = ({
                         pendingTransaction.vote?.resourceId ===
                           `${action.txHash ?? ""}${action.index ?? ""}`
                       }
-                      onClick={() => {
-                        navigate(
-                          onDashboard
-                            ? generatePath(
-                                PATHS.dashboardGovernanceActionsAction,
-                                {
-                                  proposalId: getFullGovActionId(
-                                    action.txHash,
-                                    action.index,
-                                  ),
-                                },
-                              )
-                            : PATHS.governanceActionsAction.replace(
-                                ":proposalId",
-                                getFullGovActionId(action.txHash, action.index),
-                              ),
-                          {
-                            state: {
-                              proposal: action,
-                            },
-                          },
-                        );
-                      }}
                     />
                   </div>
                 ))}
diff --git a/govtool/frontend/src/components/organisms/Slider.tsx b/govtool/frontend/src/components/organisms/Slider.tsx
@@ -1,5 +1,5 @@
-import { useCallback, useEffect, useMemo, useState } from "react";
-import { generatePath, useNavigate } from "react-router-dom";
+import { useEffect, useMemo, useState } from "react";
+import { generatePath, Link } from "react-router-dom";
 import { Box } from "@mui/material";
 import { KeenSliderOptions } from "keen-slider";
 import "keen-slider/keen-slider.min.css";
@@ -39,7 +39,6 @@ export const Slider = ({
   const [isSliderInitialized, setIsSliderInitialized] = useState(false);
 
   const { isMobile, screenWidth } = useScreenDimension();
-  const navigate = useNavigate();
   const { pendingTransaction } = useCardano();
   const { t } = useTranslation();
 
@@ -77,19 +76,6 @@ export const Slider = ({
     instanceRef.current?.moveToIdx(0);
   };
 
-  const onClickShowAll = useCallback(() => {
-    navigate(
-      generatePath(
-        onDashboard
-          ? PATHS.dashboardGovernanceActionsCategory
-          : PATHS.governanceActionsCategory,
-        {
-          category: navigateKey,
-        },
-      ),
-    );
-  }, [navigate, onDashboard]);
-
   useEffect(() => {
     if (instanceRef.current) {
       setIsSliderInitialized(true);
@@ -129,6 +115,15 @@ export const Slider = ({
           <Typography variant="title2">{title}</Typography>
           {(notSlicedDataLength > 6 || (isMobile && isShowAll)) && (
             <Button
+              component={Link}
+              to={`${generatePath(
+                  onDashboard
+                      ? PATHS.dashboardGovernanceActionsCategory
+                      : PATHS.governanceActionsCategory,
+                  {
+                    category: navigateKey,
+                  },
+              )}`}
               variant="contained"
               size="medium"
               sx={{
@@ -139,7 +134,6 @@ export const Slider = ({
                 minWidth: 93,
                 "&:hover": { backgroundColor: arcticWhite },
               }}
-              onClick={onClickShowAll}
             >
               {t("slider.showAll")}
             </Button>
diff --git a/govtool/frontend/src/components/organisms/ValidatedGovernanceActionCard.tsx b/govtool/frontend/src/components/organisms/ValidatedGovernanceActionCard.tsx
@@ -1,8 +1,8 @@
 import { useState, useEffect } from "react";
 
+import { GovernanceActionCard } from "@molecules";
 import { useValidateMutation } from "@/hooks/mutations";
 import { MetadataStandard, ProposalData } from "@/models";
-import { GovernanceActionCard } from "../molecules";
 
 type ActionTypeProps = Omit<
   ProposalData,
diff --git a/govtool/frontend/src/pages/DashboardGovernanceActionsCategory.tsx b/govtool/frontend/src/pages/DashboardGovernanceActionsCategory.tsx
@@ -1,5 +1,5 @@
 import { useMemo, useRef } from "react";
-import { generatePath, useNavigate, useParams } from "react-router-dom";
+import { useNavigate, useParams } from "react-router-dom";
 import { Box, CircularProgress, Link } from "@mui/material";
 
 import { Background, Typography } from "@atoms";
@@ -18,7 +18,6 @@ import {
   useTranslation,
 } from "@hooks";
 import {
-  getFullGovActionId,
   getProposalTypeLabel,
   removeDuplicatedProposals,
 } from "@utils";
@@ -141,18 +140,6 @@ export const DashboardGovernanceActionsCategory = () => {
                   }
                   onClick={() => {
                     saveScrollPosition();
-
-                    navigate(
-                      generatePath(PATHS.dashboardGovernanceActionsAction, {
-                        proposalId: getFullGovActionId(item.txHash, item.index),
-                      }),
-                      {
-                        state: {
-                          proposal: item,
-                          openedFromCategoryPage: true,
-                        },
-                      },
-                    );
                   }}
                   txHash={item.txHash}
                 />
diff --git a/govtool/frontend/src/pages/GovernanceActionsCategory.tsx b/govtool/frontend/src/pages/GovernanceActionsCategory.tsx
@@ -20,7 +20,6 @@ import {
 } from "@hooks";
 import {
   WALLET_LS_KEY,
-  getFullGovActionId,
   getItemFromLocalStorage,
   getProposalTypeLabel,
   removeDuplicatedProposals,
@@ -142,19 +141,6 @@ export const GovernanceActionsCategory = () => {
                         {...item}
                         onClick={() => {
                           saveScrollPosition();
-
-                          navigate(
-                            PATHS.governanceActionsAction.replace(
-                              ":proposalId",
-                              getFullGovActionId(item.txHash, item.index),
-                            ),
-                            {
-                              state: {
-                                proposal: item,
-                                openedFromCategoryPage: true,
-                              },
-                            },
-                          );
                         }}
                       />
                     </Box>
diff --git a/govtool/frontend/src/stories/Slider.stories.tsx b/govtool/frontend/src/stories/Slider.stories.tsx
@@ -58,6 +58,6 @@ export const SliderComponentOverflow: Story = {
     await expect(canvas.getByText("Slider title")).toBeInTheDocument();
     await expect(canvas.getAllByTestId("slider")).toHaveLength(6);
 
-    await expect(canvas.getByRole("button")).toBeEnabled();
+    await expect(canvas.getByRole("link")).toBeEnabled();
   },
 };
