From a768ca488f5f693efa22807d9bbebf148a7e96d7 Mon Sep 17 00:00:00 2001
From: Aaron Boyle <aaronboyle255@gmail.com>
Date: Wed, 23 Jul 2025 21:26:13 +0100
Subject: [PATCH 1/2] Revert "Fix Back button on GA category view and tests"

This reverts commit d86def3ddef27c1e6ddba9bd9a90d0a0fcf028a6.
---
 govtool/frontend/src/App.tsx                  |  5 +----
 .../frontend/src/components/atoms/types.ts    |  5 -----
 .../molecules/GovernanceActionCard.tsx        | 21 ++-----------------
 .../ValidatedGovernanceActionCard.tsx         |  3 +--
 .../DashboardGovernanceActionsCategory.tsx    | 15 ++++++++++++-
 .../src/pages/GovernanceActionsCategory.tsx   | 14 +++++++++++++
 .../frontend/src/stories/Slider.stories.tsx   |  2 +-
 7 files changed, 33 insertions(+), 32 deletions(-)

diff --git a/govtool/frontend/src/App.tsx b/govtool/frontend/src/App.tsx
index ce4f54add..f5f2fd499 100644
--- a/govtool/frontend/src/App.tsx
+++ b/govtool/frontend/src/App.tsx
@@ -81,10 +81,7 @@ export default () => {
           enable(walletName);
           return;
         }
-        // eslint-disable-next-line no-await-in-loop
-        await new Promise((resolve) => {
-          setTimeout(resolve, interval);
-        });
+        await new Promise(resolve => setTimeout(resolve, interval));
       }
 
       if (!isOnAllowedPage) {
diff --git a/govtool/frontend/src/components/atoms/types.ts b/govtool/frontend/src/components/atoms/types.ts
index 3ae4ea5ae..52bf82151 100644
--- a/govtool/frontend/src/components/atoms/types.ts
+++ b/govtool/frontend/src/components/atoms/types.ts
@@ -1,5 +1,4 @@
 import { ChangeEvent } from "react";
-import { LinkProps } from "react-router-dom";
 import {
   ButtonProps as MUIButtonProps,
   CheckboxProps as MUICheckboxProps,
@@ -14,10 +13,6 @@ 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
index 4d091020f..263ed05b3 100644
--- a/govtool/frontend/src/components/molecules/GovernanceActionCard.tsx
+++ b/govtool/frontend/src/components/molecules/GovernanceActionCard.tsx
@@ -40,7 +40,6 @@ export const GovernanceActionCard: FC<ActionTypeProps> = ({
   inProgress = false,
   expiryDate,
   expiryEpochNo,
-  onClick,
   createdDate,
   createdEpochNo,
   txHash,
@@ -59,8 +58,7 @@ export const GovernanceActionCard: FC<ActionTypeProps> = ({
     bech32Prefix: "gov_action",
   });
 
-  const pathname = useLocation().pathname.replace(/governance_actions.*/g, "governance_actions");
-  const isCategoryView = useLocation().pathname.includes("category");
+  const location = useLocation();
 
   return (
     <Box
@@ -154,23 +152,8 @@ export const GovernanceActionCard: FC<ActionTypeProps> = ({
           <Skeleton width="100%" height="40px" sx={{ borderRadius: "20px" }} />
         ) : (
           <Button
-            onClick={onClick}
             component={Link}
-            to={`${pathname}/${govActionId}`}
-            state={{
-              proposal: {
-                  abstract,
-                  type,
-                  expiryDate,
-                  expiryEpochNo,
-                  createdDate,
-                  createdEpochNo,
-                  txHash,
-                  index,
-                  title,
-              },
-              openedFromCategoryPage: isCategoryView
-            }}
+            to={`${location.pathname}/${govActionId}`}
             variant={inProgress ? "outlined" : "contained"}
             size="large"
             sx={{
diff --git a/govtool/frontend/src/components/organisms/ValidatedGovernanceActionCard.tsx b/govtool/frontend/src/components/organisms/ValidatedGovernanceActionCard.tsx
index aa37b32df..826dfb3d4 100644
--- 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,
@@ -14,7 +14,6 @@ type ActionTypeProps = Omit<
   | "rationale"
   | "motivation"
 > & {
-  onClick?: () => void;
   inProgress?: boolean;
 };
 export const ValidatedGovernanceActionCard = (props: ActionTypeProps) => {
diff --git a/govtool/frontend/src/pages/DashboardGovernanceActionsCategory.tsx b/govtool/frontend/src/pages/DashboardGovernanceActionsCategory.tsx
index a4316ed2b..fb366fc67 100644
--- a/govtool/frontend/src/pages/DashboardGovernanceActionsCategory.tsx
+++ b/govtool/frontend/src/pages/DashboardGovernanceActionsCategory.tsx
@@ -1,5 +1,5 @@
 import { useMemo, useRef } from "react";
-import { useNavigate, useParams } from "react-router-dom";
+import { generatePath, useNavigate, useParams } from "react-router-dom";
 import { Box, CircularProgress, Link } from "@mui/material";
 
 import { Background, Typography } from "@atoms";
@@ -18,6 +18,7 @@ import {
   useTranslation,
 } from "@hooks";
 import {
+  getFullGovActionId,
   getProposalTypeLabel,
   removeDuplicatedProposals,
 } from "@utils";
@@ -140,6 +141,18 @@ 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
index ace4a0125..7eda23034 100644
--- a/govtool/frontend/src/pages/GovernanceActionsCategory.tsx
+++ b/govtool/frontend/src/pages/GovernanceActionsCategory.tsx
@@ -20,6 +20,7 @@ import {
 } from "@hooks";
 import {
   WALLET_LS_KEY,
+  getFullGovActionId,
   getItemFromLocalStorage,
   getProposalTypeLabel,
   removeDuplicatedProposals,
@@ -141,6 +142,19 @@ 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
index 16ca0e518..578d8c391 100644
--- 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("link")).toBeEnabled();
+    await expect(canvas.getByRole("button")).toBeEnabled();
   },
 };

From 2a5bd884677cb94f27d4a1a37bef599f09580780 Mon Sep 17 00:00:00 2001
From: Aaron Boyle <aaronboyle255@gmail.com>
Date: Wed, 23 Jul 2025 21:27:40 +0100
Subject: [PATCH 2/2] Revert "Allow Opening Governance Actions in New Tab from
 Live Voting Page"

This reverts commit 0a50bb5e33bb8f6ed7457d485a20cc25fc79314d.
---
 govtool/frontend/src/App.tsx                  | 52 +++++++------------
 .../molecules/GovernanceActionCard.tsx        |  7 +--
 .../organisms/GovernanceActionsToVote.tsx     | 28 +++++++++-
 .../src/components/organisms/Slider.tsx       | 28 ++++++----
 .../ValidatedGovernanceActionCard.tsx         |  3 +-
 5 files changed, 66 insertions(+), 52 deletions(-)

diff --git a/govtool/frontend/src/App.tsx b/govtool/frontend/src/App.tsx
index f5f2fd499..c4903b4d8 100644
--- a/govtool/frontend/src/App.tsx
+++ b/govtool/frontend/src/App.tsx
@@ -63,44 +63,28 @@ export default () => {
   }, []);
 
   const checkTheWalletIsActive = useCallback(() => {
-    const isWalletAvailable = () =>
-        window.cardano && walletName && Object.keys(window.cardano).includes(walletName);
+    const hrefCondition =
+      window.location.pathname === PATHS.home ||
+      window.location.pathname === PATHS.governanceActions ||
+      window.location.pathname === PATHS.governanceActionsAction;
 
-    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;
-        }
-        await new Promise(resolve => setTimeout(resolve, interval));
+    const walletName = getItemFromLocalStorage(`${WALLET_LS_KEY}_name`);
+    if (window.cardano) {
+      const walletExtensions = Object.keys(window.cardano);
+      if (walletName && walletExtensions.includes(walletName)) {
+        enable(walletName);
+        return;
       }
-
-      if (!isOnAllowedPage) {
+    }
+    if (
+      (!window.cardano && walletName) ||
+      (walletName && !Object.keys(window.cardano).includes(walletName))
+    ) {
+      if (!hrefCondition) {
         navigate(PATHS.home);
       }
-      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();
+      removeItemFromLocalStorage(`${WALLET_LS_KEY}_name`);
+      removeItemFromLocalStorage(`${WALLET_LS_KEY}_stake_key`);
     }
   }, []);
 
diff --git a/govtool/frontend/src/components/molecules/GovernanceActionCard.tsx b/govtool/frontend/src/components/molecules/GovernanceActionCard.tsx
index 263ed05b3..47c98f6be 100644
--- a/govtool/frontend/src/components/molecules/GovernanceActionCard.tsx
+++ b/govtool/frontend/src/components/molecules/GovernanceActionCard.tsx
@@ -1,6 +1,5 @@
 import { FC } from "react";
 import { Box, Skeleton } from "@mui/material";
-import { Link, useLocation } from "react-router-dom";
 
 import { Button } from "@atoms";
 import {
@@ -40,6 +39,7 @@ export const GovernanceActionCard: FC<ActionTypeProps> = ({
   inProgress = false,
   expiryDate,
   expiryEpochNo,
+  onClick,
   createdDate,
   createdEpochNo,
   txHash,
@@ -58,8 +58,6 @@ export const GovernanceActionCard: FC<ActionTypeProps> = ({
     bech32Prefix: "gov_action",
   });
 
-  const location = useLocation();
-
   return (
     <Box
       sx={{
@@ -152,8 +150,7 @@ export const GovernanceActionCard: FC<ActionTypeProps> = ({
           <Skeleton width="100%" height="40px" sx={{ borderRadius: "20px" }} />
         ) : (
           <Button
-            component={Link}
-            to={`${location.pathname}/${govActionId}`}
+            onClick={onClick}
             variant={inProgress ? "outlined" : "contained"}
             size="large"
             sx={{
diff --git a/govtool/frontend/src/components/organisms/GovernanceActionsToVote.tsx b/govtool/frontend/src/components/organisms/GovernanceActionsToVote.tsx
index 332aaa0ab..768bdc609 100644
--- a/govtool/frontend/src/components/organisms/GovernanceActionsToVote.tsx
+++ b/govtool/frontend/src/components/organisms/GovernanceActionsToVote.tsx
@@ -1,10 +1,12 @@
+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 } from "@utils";
+import { getProposalTypeTitle, getFullGovActionId } from "@utils";
 import { Slider, ValidatedGovernanceActionCard } from "@organisms";
 
 type GovernanceActionsToVoteProps = {
@@ -23,6 +25,7 @@ export const GovernanceActionsToVote = ({
   sorting,
 }: GovernanceActionsToVoteProps) => {
   const { pendingTransaction } = useCardano();
+  const navigate = useNavigate();
   const { isMobile, pagePadding } = useScreenDimension();
   const { t } = useTranslation();
 
@@ -61,6 +64,29 @@ 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
index af1b745d8..cee7abc27 100644
--- a/govtool/frontend/src/components/organisms/Slider.tsx
+++ b/govtool/frontend/src/components/organisms/Slider.tsx
@@ -1,5 +1,5 @@
-import { useEffect, useMemo, useState } from "react";
-import { generatePath, Link } from "react-router-dom";
+import { useCallback, useEffect, useMemo, useState } from "react";
+import { generatePath, useNavigate } from "react-router-dom";
 import { Box } from "@mui/material";
 import { KeenSliderOptions } from "keen-slider";
 import "keen-slider/keen-slider.min.css";
@@ -39,6 +39,7 @@ export const Slider = ({
   const [isSliderInitialized, setIsSliderInitialized] = useState(false);
 
   const { isMobile, screenWidth } = useScreenDimension();
+  const navigate = useNavigate();
   const { pendingTransaction } = useCardano();
   const { t } = useTranslation();
 
@@ -76,6 +77,19 @@ 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);
@@ -115,15 +129,6 @@ 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={{
@@ -134,6 +139,7 @@ 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
index 826dfb3d4..4dfe5ae0a 100644
--- a/govtool/frontend/src/components/organisms/ValidatedGovernanceActionCard.tsx
+++ b/govtool/frontend/src/components/organisms/ValidatedGovernanceActionCard.tsx
@@ -2,7 +2,7 @@ import { useState, useEffect } from "react";
 
 import { useValidateMutation } from "@/hooks/mutations";
 import { MetadataStandard, ProposalData } from "@/models";
-import { GovernanceActionCard } from "@molecules";
+import { GovernanceActionCard } from "../molecules";
 
 type ActionTypeProps = Omit<
   ProposalData,
@@ -14,6 +14,7 @@ type ActionTypeProps = Omit<
   | "rationale"
   | "motivation"
 > & {
+  onClick?: () => void;
   inProgress?: boolean;
 };
 export const ValidatedGovernanceActionCard = (props: ActionTypeProps) => {