proper fix

appventure-nush · Nov 1, 2024 · bdb8655 · bdb8655
1 parent ac4252e
commit bdb8655
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/backend/src/api/admin.ts b/backend/src/api/admin.ts
@@ -73,11 +73,13 @@ router.post("/addMoney", async (req, res) => {
         return res.status(400).json({ message: "Amount must be a string" })
     }
 
-    if (Number(amount) <= 0) {
+    const amountToDP = new Decimal(amount).toDP(2)
+
+    if (amountToDP.lte(0)) {
         return res.status(400).json({ message: "Amount must be positive" })
     }
 
-    if (!isFinite(Number(amount))) {
+    if (!amountToDP.isFinite()) {
         return res.status(400).json({ message: "No scamming" })
     }
 
@@ -97,8 +99,6 @@ router.post("/addMoney", async (req, res) => {
             return res.status(400).json({ message: "Token ID already used" })
         }
 
-        const amountToDP = new Decimal(amount).toDP(2)
-
         // update topup data
         topupRow.admin_uid = req.user!.uid
         topupRow.admin_name = req.user!.name