From 0e1216b8e18fd501e056dd3738b708d456b59cb9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Sza=C5=82owski?= <michal.szalowski@gmail.com>
Date: Tue, 27 Feb 2024 10:54:10 +0100
Subject: [PATCH] [#119] Post code review adjustments

---
 .../src/components/molecules/DashboardActionCard.tsx       | 5 +++--
 .../frontend/src/components/molecules/VoteActionForm.tsx   | 2 +-
 govtool/frontend/src/pages/Dashboard.tsx                   | 7 ++-----
 govtool/frontend/src/pages/DelegateTodRep.tsx              | 7 ++-----
 govtool/frontend/src/pages/RegisterAsSoleVoter.tsx         | 7 ++-----
 govtool/frontend/src/pages/RegisterAsdRep.tsx              | 7 ++-----
 govtool/frontend/src/pages/RetireAsSoleVoter.tsx           | 7 ++-----
 govtool/frontend/src/pages/UpdatedRepMetadata.tsx          | 7 ++-----
 govtool/frontend/src/utils/checkIsWalletConnected.ts       | 5 +++++
 govtool/frontend/src/utils/index.ts                        | 3 ++-
 10 files changed, 23 insertions(+), 34 deletions(-)
 create mode 100644 govtool/frontend/src/utils/checkIsWalletConnected.ts

diff --git a/govtool/frontend/src/components/molecules/DashboardActionCard.tsx b/govtool/frontend/src/components/molecules/DashboardActionCard.tsx
index 48bd969ec..d394c12a4 100644
--- a/govtool/frontend/src/components/molecules/DashboardActionCard.tsx
+++ b/govtool/frontend/src/components/molecules/DashboardActionCard.tsx
@@ -2,7 +2,7 @@ import { Box, ButtonProps, Skeleton } from "@mui/material";
 import { FC, ReactNode } from "react";
 
 import { CopyButton, LoadingButton, Typography } from "@atoms";
-import { useScreenDimension } from "@hooks";
+import { useScreenDimension, useTranslation } from "@hooks";
 import { theme } from "@/theme";
 
 type DashboardActionCardProps = {
@@ -31,6 +31,7 @@ type DashboardActionCardProps = {
 export const DashboardActionCard: FC<DashboardActionCardProps> = ({
   ...props
 }) => {
+  const { t } = useTranslation();
   const {
     cardId,
     cardTitle,
@@ -83,7 +84,7 @@ export const DashboardActionCard: FC<DashboardActionCardProps> = ({
           }}
         >
           <Typography color={"orangeDark"} variant="body2">
-            In progress
+            {t("inProgress").toLocaleUpperCase()}
           </Typography>
         </Box>
       )}
diff --git a/govtool/frontend/src/components/molecules/VoteActionForm.tsx b/govtool/frontend/src/components/molecules/VoteActionForm.tsx
index 211d6ad50..920021f24 100644
--- a/govtool/frontend/src/components/molecules/VoteActionForm.tsx
+++ b/govtool/frontend/src/components/molecules/VoteActionForm.tsx
@@ -142,7 +142,7 @@ export const VoteActionForm = ({
             />
           </Box>
         </Box>
-        {voter?.isRegisteredAsDRep && (
+        {(voter?.isRegisteredAsDRep || voter?.isRegisteredAsSoleVoter) && (
           <Button
             data-testid="show-votes-button"
             variant="text"
diff --git a/govtool/frontend/src/pages/Dashboard.tsx b/govtool/frontend/src/pages/Dashboard.tsx
index db72fd2a2..7899270cc 100644
--- a/govtool/frontend/src/pages/Dashboard.tsx
+++ b/govtool/frontend/src/pages/Dashboard.tsx
@@ -7,7 +7,7 @@ import { useCardano } from "@context";
 import { Background, ScrollToManage } from "@atoms";
 import { useScreenDimension } from "@hooks";
 import { DashboardTopNav, Drawer, Footer } from "@organisms";
-import { WALLET_LS_KEY, getItemFromLocalStorage } from "@/utils/localStorage";
+import { checkIsWalletConnected } from "@/utils";
 
 const getPageTitle = (pathname: string) => {
   if (pathname === PATHS.dashboard) {
@@ -33,10 +33,7 @@ export const Dashboard = () => {
   }, [pathname, divRef]);
 
   useEffect(() => {
-    if (
-      !getItemFromLocalStorage(`${WALLET_LS_KEY}_stake_key`) ||
-      !getItemFromLocalStorage(`${WALLET_LS_KEY}_name`)
-    ) {
+    if (checkIsWalletConnected()) {
       if (window.location.pathname === PATHS.dashboard) {
         navigate(PATHS.home);
       } else {
diff --git a/govtool/frontend/src/pages/DelegateTodRep.tsx b/govtool/frontend/src/pages/DelegateTodRep.tsx
index 8749bb225..05b11cc44 100644
--- a/govtool/frontend/src/pages/DelegateTodRep.tsx
+++ b/govtool/frontend/src/pages/DelegateTodRep.tsx
@@ -10,8 +10,8 @@ import {
   Footer,
 } from "@organisms";
 import { useScreenDimension, useTranslation } from "@hooks";
-import { WALLET_LS_KEY, getItemFromLocalStorage } from "@/utils/localStorage";
 import { useNavigate } from "react-router-dom";
+import { checkIsWalletConnected } from "@/utils";
 
 export const DelegateTodRep = () => {
   const [step, setStep] = useState(1);
@@ -20,10 +20,7 @@ export const DelegateTodRep = () => {
   const { t } = useTranslation();
 
   useEffect(() => {
-    if (
-      !getItemFromLocalStorage(`${WALLET_LS_KEY}_stake_key`) ||
-      !getItemFromLocalStorage(`${WALLET_LS_KEY}_name`)
-    ) {
+    if (checkIsWalletConnected()) {
       navigate(PATHS.home);
     }
   }, []);
diff --git a/govtool/frontend/src/pages/RegisterAsSoleVoter.tsx b/govtool/frontend/src/pages/RegisterAsSoleVoter.tsx
index ea7018a85..822d809ce 100644
--- a/govtool/frontend/src/pages/RegisterAsSoleVoter.tsx
+++ b/govtool/frontend/src/pages/RegisterAsSoleVoter.tsx
@@ -4,18 +4,15 @@ import { useNavigate } from "react-router-dom";
 import { PATHS } from "@consts";
 import { RegisterAsSoleVoterBox } from "@organisms";
 import { useTranslation } from "@hooks";
-import { WALLET_LS_KEY, getItemFromLocalStorage } from "@/utils/localStorage";
 import { CenteredBoxPageWrapper } from "@molecules";
+import { checkIsWalletConnected } from "@/utils";
 
 export const RegisterAsSoleVoter = () => {
   const navigate = useNavigate();
   const { t } = useTranslation();
 
   useEffect(() => {
-    if (
-      !getItemFromLocalStorage(`${WALLET_LS_KEY}_stake_key`) ||
-      !getItemFromLocalStorage(`${WALLET_LS_KEY}_name`)
-    ) {
+    if (checkIsWalletConnected()) {
       navigate(PATHS.home);
     }
   }, []);
diff --git a/govtool/frontend/src/pages/RegisterAsdRep.tsx b/govtool/frontend/src/pages/RegisterAsdRep.tsx
index 791c30feb..3653b806d 100644
--- a/govtool/frontend/src/pages/RegisterAsdRep.tsx
+++ b/govtool/frontend/src/pages/RegisterAsdRep.tsx
@@ -17,7 +17,7 @@ import {
   useTranslation,
 } from "@hooks";
 import { useNavigate } from "react-router-dom";
-import { WALLET_LS_KEY, getItemFromLocalStorage } from "@/utils/localStorage";
+import { checkIsWalletConnected } from "@/utils";
 
 export const RegisterAsdRep = () => {
   const [step, setStep] = useState<number>(1);
@@ -28,10 +28,7 @@ export const RegisterAsdRep = () => {
   const registerAsdRepFormMethods = useRegisterAsdRepFormController();
 
   useEffect(() => {
-    if (
-      !getItemFromLocalStorage(`${WALLET_LS_KEY}_stake_key`) ||
-      !getItemFromLocalStorage(`${WALLET_LS_KEY}_name`)
-    ) {
+    if (checkIsWalletConnected()) {
       navigate(PATHS.home);
     }
   }, []);
diff --git a/govtool/frontend/src/pages/RetireAsSoleVoter.tsx b/govtool/frontend/src/pages/RetireAsSoleVoter.tsx
index c661be2cd..7b8fde1cc 100644
--- a/govtool/frontend/src/pages/RetireAsSoleVoter.tsx
+++ b/govtool/frontend/src/pages/RetireAsSoleVoter.tsx
@@ -4,18 +4,15 @@ import { PATHS } from "@consts";
 import { RetireAsSoleVoterBox } from "@organisms";
 import { useTranslation } from "@hooks";
 import { useNavigate } from "react-router-dom";
-import { WALLET_LS_KEY, getItemFromLocalStorage } from "@/utils/localStorage";
 import { CenteredBoxPageWrapper } from "@molecules";
+import { checkIsWalletConnected } from "@/utils";
 
 export const RetireAsSoleVoter = () => {
   const navigate = useNavigate();
   const { t } = useTranslation();
 
   useEffect(() => {
-    if (
-      !getItemFromLocalStorage(`${WALLET_LS_KEY}_stake_key`) ||
-      !getItemFromLocalStorage(`${WALLET_LS_KEY}_name`)
-    ) {
+    if (checkIsWalletConnected()) {
       navigate(PATHS.home);
     }
   }, []);
diff --git a/govtool/frontend/src/pages/UpdatedRepMetadata.tsx b/govtool/frontend/src/pages/UpdatedRepMetadata.tsx
index 6d5b0cb22..3d80064e7 100644
--- a/govtool/frontend/src/pages/UpdatedRepMetadata.tsx
+++ b/govtool/frontend/src/pages/UpdatedRepMetadata.tsx
@@ -12,7 +12,7 @@ import {
 } from "@hooks";
 import { ControlledField, DashboardTopNav, Footer } from "@organisms";
 import { theme } from "@/theme";
-import { WALLET_LS_KEY, getItemFromLocalStorage, openInNewTab } from "@utils";
+import { checkIsWalletConnected, openInNewTab } from "@utils";
 
 export const UpdatedRepMetadata = () => {
   const navigate = useNavigate();
@@ -27,10 +27,7 @@ export const UpdatedRepMetadata = () => {
     useUpdatedRepMetadataForm();
 
   useEffect(() => {
-    if (
-      !getItemFromLocalStorage(`${WALLET_LS_KEY}_stake_key`) ||
-      !getItemFromLocalStorage(`${WALLET_LS_KEY}_name`)
-    ) {
+    if (checkIsWalletConnected()) {
       navigate(PATHS.home);
     } else {
       const isPendingTx = isPendingTransaction();
diff --git a/govtool/frontend/src/utils/checkIsWalletConnected.ts b/govtool/frontend/src/utils/checkIsWalletConnected.ts
new file mode 100644
index 000000000..4c91a1137
--- /dev/null
+++ b/govtool/frontend/src/utils/checkIsWalletConnected.ts
@@ -0,0 +1,5 @@
+import { WALLET_LS_KEY, getItemFromLocalStorage } from "@/utils/localStorage";
+
+export const checkIsWalletConnected = () =>
+  !getItemFromLocalStorage(`${WALLET_LS_KEY}_stake_key`) ||
+  !getItemFromLocalStorage(`${WALLET_LS_KEY}_name`);
diff --git a/govtool/frontend/src/utils/index.ts b/govtool/frontend/src/utils/index.ts
index f28259282..95c368dd8 100644
--- a/govtool/frontend/src/utils/index.ts
+++ b/govtool/frontend/src/utils/index.ts
@@ -2,7 +2,7 @@ export * from "./adaFormat";
 export * from "./basicReducer";
 export * from "./callAll";
 export * from "./formatDate";
-export * from "./getDRepId";
+export * from "./getDRepID";
 export * from "./getLengthInBytes";
 export * from "./getProposalTypeLabel";
 export * from "./isValidFormat";
@@ -11,3 +11,4 @@ export * from "./openInNewTab";
 export * from "./getGovActionId";
 export * from "./checkIsMaintenanceOn";
 export * from "./removeDuplicatedProposals";
+export * from "./checkIsWalletConnected";