Merge pull request #24 from Levana-Protocol/perp-4083/potential-earnings

PERP-4083 | Fix potential earnings microunits
Levana-Protocol · Sep 18, 2024 · 92b46eb · 92b46eb
2 parents bf29c96 + 05942f7
commit 92b46eb
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/frontend/src/utils/coins.ts b/frontend/src/utils/coins.ts
@@ -315,6 +315,11 @@ class Coins extends Asset {
     const newUnits = this.units.times(value)
     return Coins.fromUnits(this.denom, newUnits)
   }
+
+  dividedBy(value: BigNumber.Value): Coins {
+    const newUnits = this.units.dividedBy(value)
+    return Coins.fromUnits(this.denom, newUnits)
+  }
 }
 
 class USD extends Asset {

diff --git a/frontend/src/utils/shares.ts b/frontend/src/utils/shares.ts
@@ -2,7 +2,7 @@ import BigNumber from "bignumber.js"
 
 import type { Market, OutcomeId } from "@api/queries/Market"
 import type { Positions } from "@api/queries/Positions"
-import type { Coins } from "./coins"
+import { Coins } from "./coins"
 
 class Shares {
   static precision = 3
@@ -46,7 +46,10 @@ const getShares = (positions: Positions, outcomeId: OutcomeId): Shares => {
 }
 
 const getPotentialWinnings = (market: Market, positionSize: Shares): Coins => {
-  return market.poolSize.times(positionSize.value)
+  return Coins.fromUnits(
+    market.denom,
+    market.poolSize.getValue().times(positionSize.value),
+  )
 }
 
 export { Shares, getShares, getPotentialWinnings }