Merge pull request #108 from valory-xyz/tanya/registry-gas-fix

(registry) fix: add gas estimation to creating agent and component
valory-xyz · Oct 1, 2024 · 84158ce · 84158ce
2 parents a1bf4aa + 7fb85ce
commit 84158ce
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 16 deletions.
diff --git a/apps/autonolas-registry/components/ListAgents/mint.jsx b/apps/autonolas-registry/components/ListAgents/mint.jsx
@@ -3,6 +3,8 @@ import { useRouter } from 'next/router';
 import { Typography } from 'antd';
 import { notifyError, notifySuccess } from '@autonolas/frontend-library';
 
+import { getEstimatedGasLimit } from 'libs/util-functions/src';
+
 import RegisterForm from 'common-util/List/RegisterForm';
 import { AlertSuccess, AlertError } from 'common-util/List/ListCommon';
 import { getMechMinterContract } from 'common-util/Contracts';
@@ -41,14 +43,14 @@ const MintAgent = () => {
 
       const contract = getMechMinterContract(account);
 
-      const fn = contract.methods
-        .create(
-          '1',
-          values.owner_address,
-          `0x${values.hash}`,
-          values.dependencies ? values.dependencies.split(', ') : [],
-        )
-        .send({ from: account });
+      const createFn = contract.methods.create(
+        '1',
+        values.owner_address,
+        `0x${values.hash}`,
+        values.dependencies ? values.dependencies.split(', ') : [],
+      );
+      const estimatedGas = await getEstimatedGasLimit(createFn, account);
+      const fn = createFn.send({ from: account, gasLimit: estimatedGas });
 
       sendTransaction(fn, account)
         .then((result) => {

diff --git a/apps/autonolas-registry/components/ListComponents/mint.jsx b/apps/autonolas-registry/components/ListComponents/mint.jsx
@@ -3,6 +3,8 @@ import { useRouter } from 'next/router';
 import { Typography } from 'antd';
 import { notifyError, notifySuccess } from '@autonolas/frontend-library';
 
+import { getEstimatedGasLimit } from 'libs/util-functions/src';
+
 import RegisterForm from 'common-util/List/RegisterForm';
 import { AlertSuccess, AlertError } from 'common-util/List/ListCommon';
 import { getMechMinterContract } from 'common-util/Contracts';
@@ -40,14 +42,14 @@ const MintComponent = () => {
       }
 
       const contract = getMechMinterContract();
-      const fn = contract.methods
-        .create(
-          '0',
-          values.owner_address,
-          `0x${values.hash}`,
-          values.dependencies ? values.dependencies.split(', ') : [],
-        )
-        .send({ from: account });
+      const createFn = contract.methods.create(
+        '0',
+        values.owner_address,
+        `0x${values.hash}`,
+        values.dependencies ? values.dependencies.split(', ') : [],
+      );
+      const estimatedGas = await getEstimatedGasLimit(createFn, account);
+      const fn = createFn.send({ from: account, gasLimit: estimatedGas });
 
       sendTransaction(fn, account)
         .then((result) => {