fix: skip admin transfer in deployment scripts when admin is deployer (…

…#457)
zeta-chain · Jan 31, 2025 · cc6041c · cc6041c
1 parent 4b8c89f
commit cc6041c
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 5 deletions.
diff --git a/scripts/deploy/deterministic/DeployERC20Custody.s.sol b/scripts/deploy/deterministic/DeployERC20Custody.s.sol
@@ -60,7 +60,9 @@ contract DeployERC20Custody is Script {
         require(address(erc20Custody.gateway()) == gateway, "gateway not set");
 
         // Transfer admin role from deployer to admin
-        transferAdmin(erc20Custody, msg.sender, admin);
+        if (msg.sender != admin) {
+            transferAdmin(erc20Custody, msg.sender, admin);
+        }
 
         vm.stopBroadcast();
     }

diff --git a/scripts/deploy/deterministic/DeployGatewayEVM.s.sol b/scripts/deploy/deterministic/DeployGatewayEVM.s.sol
@@ -60,9 +60,10 @@ contract DeployGatewayEVM is Script {
         require(gateway.tssAddress() == tss, "tss not set");
         require(gateway.zetaToken() == address(zeta), "zeta token not set");
 
-        // Transfer admin role from deployer to admin
-        transferAdmin(gateway, msg.sender, admin);
-
+        // Transfer admin role from deployer to admin if the address is different
+        if (msg.sender != admin) {
+            transferAdmin(gateway, msg.sender, admin);
+        }
         vm.stopBroadcast();
     }
 

diff --git a/scripts/deploy/deterministic/DeployGatewayZEVM.s.sol b/scripts/deploy/deterministic/DeployGatewayZEVM.s.sol
@@ -58,7 +58,9 @@ contract DeployGatewayZEVM is Script {
         require(gateway.zetaToken() == zeta, "zeta token not set");
 
         // Transfer admin role from deployer to admin
-        transferAdmin(gateway, msg.sender, admin);
+        if (msg.sender != admin) {
+            transferAdmin(gateway, msg.sender, admin);
+        }
 
         vm.stopBroadcast();
     }