deploy: script: TransferOwernship: wait for prev transaction before initiating next

Author: apoorvlathey

deploy/scripts/TransferOwnership.ts

@@ -10,43 +10,42 @@ const func: DeployFunction = async function (hre: HardhatRuntimeEnvironment) {
     return;
   }
 
-  await Promise.all(
-    [
-      "DefaultProxyAdmin",
-      "MarketplaceUniversalRouterZap",
-      "MigratorZap",
-      "FailSafe",
-      "NFTXFeeDistributorV3",
-      "NFTXInventoryStakingV3Upgradeable",
-      "NFTXRouter",
-      "NFTXVaultFactoryUpgradeableV3",
-      "NonfungiblePositionManager",
-      "UniswapV3FactoryUpgradeable",
-    ].map(async (name) => {
-      const contract = await getContract(hre, name);
-      const owner = await contract.owner();
+  // sequentially transfer ownership of contracts
+  const contractNames = [
+    "DefaultProxyAdmin",
+    "MarketplaceUniversalRouterZap",
+    "MigratorZap",
+    "FailSafe",
+    "NFTXFeeDistributorV3",
+    "NFTXInventoryStakingV3Upgradeable",
+    "NFTXRouter",
+    "NFTXVaultFactoryUpgradeableV3",
+    "NonfungiblePositionManager",
+    "UniswapV3FactoryUpgradeable",
+  ];
 
-      if (owner.toLowerCase() === config.multisig.toLowerCase()) {
-        console.log(`Ownership of ${name} already transferred`);
-      } else {
-        try {
-          console.log(`⌛ Transferring ownership of ${name}...`);
+  for (const name of contractNames) {
+    const contract = await getContract(hre, name);
+    const owner = await contract.owner();
 
-          await execute(
-            name,
-            { from: deployer },
-            "transferOwnership",
-            config.multisig
-          );
-
-          console.log(`✅ Ownership of ${name} transferred`);
-        } catch (e) {
-          console.log(`🚨 Failed to transfer ownership of ${name}`);
-          console.log(e);
-        }
+    if (owner.toLowerCase() === config.multisig.toLowerCase()) {
+      console.log(`Ownership of ${name} already transferred`);
+    } else {
+      try {
+        console.log(`⌛ Transferring ownership of ${name}...`);
+        await execute(
+          name,
+          { from: deployer },
+          "transferOwnership",
+          config.multisig
+        );
+        console.log(`✅ Ownership of ${name} transferred`);
+      } catch (e) {
+        console.log(`🚨 Failed to transfer ownership of ${name}`);
+        console.log(e);
       }
-    })
-  );
+    }
+  }
 };
 export default func;
 func.tags = ["TransferOwnership"];