Merge pull request #1174 from Vayras/ModalBug

removed the small white module that appeared when clicking on bounty #1029
stakwork · Dec 26, 2023 · 34601cf · 34601cf
2 parents 0379c11 + 25081aa
commit 34601cf
Showing 1 changed file with 13 additions and 2 deletions.
diff --git a/frontend/app/src/people/main/bountyModal/BountyModal.tsx b/frontend/app/src/people/main/bountyModal/BountyModal.tsx
@@ -21,6 +21,7 @@ export const BountyModal = ({ basePath, fromPage, bountyOwner }: BountyModalProp
   const { person } = usePerson(ui.selectedPerson);
   const [bounty, setBounty] = useState<PersonBounty[]>([]);
   const [afterEdit, setAfterEdit] = useState(false);
+  const [loading, setLoading] = useState(true);
 
   const personToDisplay = fromPage === 'usertickets' ? bountyOwner : person;
 
@@ -41,8 +42,14 @@ export const BountyModal = ({ basePath, fromPage, bountyOwner }: BountyModalProp
        * infinite loop and crash the app
        */
       if ((wantedId && !bounty.length) || afterEdit) {
-        const bounty = await main.getBountyById(Number(wantedId));
-        setBounty(bounty);
+        try {
+          const bountyData = await main.getBountyById(Number(wantedId));
+          setBounty(bountyData);
+        } catch (error) {
+          console.error('Error fetching bounty:', error);
+        } finally {
+          setLoading(false);
+        }
       }
     },
     [bounty, main, wantedId]
@@ -61,6 +68,10 @@ export const BountyModal = ({ basePath, fromPage, bountyOwner }: BountyModalProp
 
   const isMobile = useIsMobile();
 
+  if (loading) {
+    return null;
+  }
+
   if (isMobile) {
     return (
       <Modal visible={true} fill={true}>