fix: Fix sdk decimal issue

bnb-chain · Dec 20, 2024 · 5414b01 · 5414b01
1 parent 745c29c
commit 5414b01
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/packages/canonical-bridge-sdk/src/core/utils/number.ts b/packages/canonical-bridge-sdk/src/core/utils/number.ts
@@ -12,10 +12,15 @@ export function removeAfterDecimals(num: string | number, decimals = 8) {
   return result;
 }
 
-export const formatNumber = (value: number, decimals = 18) => {
+export const formatNumber = (
+  value: number,
+  decimals = 18,
+  useGrouping = true
+) => {
   const num = removeAfterDecimals(value, decimals);
   return num.toLocaleString('fullwide', {
     maximumFractionDigits: decimals,
+    useGrouping,
   });
 };
 

diff --git a/packages/canonical-bridge-sdk/src/layerZero/index.ts b/packages/canonical-bridge-sdk/src/layerZero/index.ts
@@ -80,7 +80,7 @@ export class LayerZero {
       ];
       const nativeFee = fees[0];
       const minAmount = parseUnits(
-        String(formatNumber(Number(formatUnits(amount, 18)), 8)),
+        String(formatNumber(Number(formatUnits(amount, 18)), 8, false)),
         18
       );
       const cakeArgs = {