Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit e1108eb

Browse files
committedMay 24, 2025·
chore: remove added code
1 parent 6f66fc6 commit e1108eb

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed
 

‎script/releases/v.1.5.0-redistribution/1-deployContracts.s.sol

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -260,7 +260,7 @@ contract Deploy is EOADeployer {
260260
);
261261
}
262262

263-
// SlashEscrowProxyAdmin - proxyAdmin is the community multisig
263+
// SlashEscrowProxyAdmin - proxyAdmin is unique and its admin is the community multisig
264264
assertTrue(
265265
_getSlashEscrowProxyAdmin(address(Env.proxy.slashEscrowFactory())) == slashEscrowProxyAdmin,
266266
"slashEscrowFactory proxyAdmin incorrect"

‎src/contracts/permissions/PermissionController.sol

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -61,15 +61,15 @@ contract PermissionController is Initializable, SemVerMixin, PermissionControlle
6161
function acceptAdmin(
6262
address account
6363
) external {
64-
// AccountPermissions storage permissions = _permissions[account];
64+
AccountPermissions storage permissions = _permissions[account];
6565

6666
// Remove the admin from the pending list
6767
// Revert if the admin is not pending
68-
require(_permissions[account].pendingAdmins.remove(msg.sender), AdminNotPending());
68+
require(permissions.pendingAdmins.remove(msg.sender), AdminNotPending());
6969

7070
// Add the admin to the account's admins
7171
// Not wrapped in a require since it must be the case the admin is not one
72-
_permissions[account].admins.add(msg.sender);
72+
permissions.admins.add(msg.sender);
7373

7474
emit AdminSet(account, msg.sender);
7575
}

0 commit comments

Comments
 (0)
Please sign in to comment.