[#80] wrap with useCallback registration card texts

IntersectMBO · Feb 7, 2024 · dace096 · dace096
1 parent 6d21f03
commit dace096
Showing 1 changed file with 16 additions and 6 deletions.
diff --git a/govtool/frontend/src/components/organisms/DashboardCards.tsx b/govtool/frontend/src/components/organisms/DashboardCards.tsx
@@ -213,7 +213,7 @@ export const DashboardCards = () => {
     formHexToBech32,
   ]);
 
-  const getRegistrationCardDescription = () => {
+  const registrationCardDescription = useMemo(() => {
     if (registerTransaction.transactionHash) {
       switch (registerTransaction.type) {
         case "retirement":
@@ -228,9 +228,14 @@ export const DashboardCards = () => {
     } else {
       return t("dashboard.registration.ifYouWant");
     }
-  };
+  }, [
+    registerTransaction.transactionHash,
+    registerTransaction.type,
+    dRep?.isRegistered,
+    dRep?.wasRegistered,
+  ]);
 
-  const getRegistrationCardTitle = () => {
+  const registrationCardTitle = useMemo(() => {
     if (registerTransaction?.transactionHash) {
       switch (registerTransaction.type) {
         case "retirement":
@@ -247,7 +252,12 @@ export const DashboardCards = () => {
     } else {
       return t("dashboard.registration.registerAsDRep");
     }
-  };
+  }, [
+    registerTransaction?.transactionHash,
+    registerTransaction.type,
+    dRep?.isRegistered,
+    dRep?.wasRegistered,
+  ]);
 
   const renderGovActionSection = useCallback(() => {
     return (
@@ -380,7 +390,7 @@ export const DashboardCards = () => {
               ? "change-metadata-button"
               : "register-learn-more-button"
           }
-          description={getRegistrationCardDescription()}
+          description={registrationCardDescription}
           firstButtonAction={
             dRep?.isRegistered
               ? retireAsDrep
@@ -421,7 +431,7 @@ export const DashboardCards = () => {
           cardTitle={
             dRep?.isRegistered || dRep?.wasRegistered ? t("myDRepId") : ""
           }
-          title={getRegistrationCardTitle()}
+          title={registrationCardTitle}
         />
       </Box>
       {!dRep?.isRegistered && renderGovActionSection()}