Merge pull request #298 from terra-money/fix/osmosis-gas-query

feat: estimate osmosis gas at run time
stationmoney · Dec 26, 2023 · 90a7117 · 90a7117
2 parents b11972d + d4b097e
commit 90a7117
Show file tree

Hide file tree

Showing 4 changed files with 49 additions and 8 deletions.
diff --git a/public/manifest.json b/public/manifest.json
@@ -1,8 +1,8 @@
 {
   "manifest_version": 3,
   "name": "Station Wallet",
-  "version": "7.4.12",
-  "version_name": "7.4.12",
+  "version": "7.4.13",
+  "version_name": "7.4.13",
   "background": {
     "service_worker": "background.js"
   },

diff --git a/src/data/queries/tx.ts b/src/data/queries/tx.ts
@@ -1,8 +1,9 @@
 import { QueryKey, useQuery, useQueryClient } from "react-query"
 import { atom, useSetRecoilState } from "recoil"
+import axios from "axios"
+import { useNetworks } from "app/InitNetworks"
 import { queryKey } from "../query"
 import { useInterchainLCDClient } from "./lcdClient"
-import axios from "axios"
 import { RefetchOptions } from "../query"
 import { CARBON_API } from "config/constants"
 
@@ -14,6 +15,10 @@ interface LatestTx {
   onSuccess?: () => void
 }
 
+interface OsmosisGasResponse {
+  base_fee: string
+}
+
 export const latestTxState = atom<LatestTx>({
   key: "latestTx",
   default: { txhash: "", chainID: "" },
@@ -50,6 +55,31 @@ export const useTxInfo = ({ txhash, queryKeys, chainID }: LatestTx) => {
   )
 }
 
+export const useOsmosisGas = () => {
+  const { networks } = useNetworks()
+
+  return useQuery(
+    [queryKey.tx.osmosisGas],
+    async () => {
+      try {
+        const { data } = await axios.get<OsmosisGasResponse>(
+          "osmosis/txfees/v1beta1/cur_eip_base_fee",
+          {
+            baseURL: networks["mainnet"]["osmosis-1"].lcd, // hard set for now.
+          }
+        )
+        return Number(data.base_fee)
+      } catch (e) {
+        return 0.0025
+      }
+    },
+    {
+      ...RefetchOptions.INFINITY,
+      staleTime: 60 * 1000, // cache data for 1 min
+    }
+  )
+}
+
 export const useCarbonFees = () => {
   return useQuery(
     [queryKey.tx.fees],

diff --git a/src/data/query.ts b/src/data/query.ts
@@ -74,7 +74,7 @@ export const queryKey = mirror({
     unbondings: "",
     pool: "",
   },
-  tx: { txInfo: "", create: "", fees: "" },
+  tx: { txInfo: "", create: "", fees: "", osmosisGas: "" },
   wasm: { contractInfo: "", contractQuery: "" },
   interchain: {
     staking: {

diff --git a/src/txs/Tx.tsx b/src/txs/Tx.tsx
@@ -21,7 +21,11 @@ import { getLocalSetting, SettingKey } from "utils/localStorage"
 import { combineState, RefetchOptions } from "data/query"
 import { queryKey } from "data/query"
 import { useNetwork } from "data/wallet"
-import { isBroadcastingState, latestTxState } from "data/queries/tx"
+import {
+  isBroadcastingState,
+  latestTxState,
+  useOsmosisGas,
+} from "data/queries/tx"
 import { useIsWalletEmpty } from "data/queries/bank"
 
 import { Pre } from "components/general"
@@ -102,6 +106,7 @@ function Tx<TxValues>(props: Props<TxValues>) {
   const isBroadcasting = useRecoilValue(isBroadcastingState)
   const readNativeDenom = useNativeDenoms()
   const { data: carbonFees } = useCarbonFees()
+  const { data: osmosisGas } = useOsmosisGas()
 
   /* taxes */
   const isClassic = networks[chain]?.isClassic
@@ -145,6 +150,7 @@ function Tx<TxValues>(props: Props<TxValues>) {
               carbonFees?.costs["default_fee"]
           )
         }
+
         const unsignedTx = await lcd.tx.create([{ address: key.address }], {
           ...simulationTx,
           feeDenoms: [gasDenom],
@@ -171,14 +177,18 @@ function Tx<TxValues>(props: Props<TxValues>) {
     (denom: CoinDenom) => {
       const gasPrice = chain?.startsWith("carbon-")
         ? carbonFees?.prices[denom]
+        : chain?.startsWith("osmosis")
+        ? (osmosisGas || 0.0025) * 10
         : networks[chain]?.gasPrices[denom]
+
       if (isNil(estimatedGas) || !gasPrice) return "0"
+
       return new BigNumber(estimatedGas)
         .times(gasPrice)
         .integerValue(BigNumber.ROUND_CEIL)
         .toString()
     },
-    [estimatedGas, chain, networks, carbonFees]
+    [chain, carbonFees?.prices, osmosisGas, networks, estimatedGas]
   )
 
   const gasAmount = getGasAmount(gasDenom)
@@ -343,7 +353,6 @@ function Tx<TxValues>(props: Props<TxValues>) {
 
   const renderFee = (descriptions?: Contents) => {
     if (!estimatedGas) return null
-
     return (
       <Details>
         <dl>
@@ -378,7 +387,9 @@ function Tx<TxValues>(props: Props<TxValues>) {
           <dd>
             {gasFee.amount && (
               <Read
-                decimals={decimals}
+                decimals={
+                  gasDenom ? readNativeDenom(gasDenom).decimals : decimals
+                }
                 {...gasFee}
                 denom={
                   gasFee.denom === token