Skip to content

Commit

Permalink
Remove redundant checks (#113)
Browse files Browse the repository at this point in the history
close #97
  • Loading branch information
sirnicolaz authored Aug 11, 2023
1 parent 80e2776 commit d3aaa63
Showing 1 changed file with 0 additions and 21 deletions.
21 changes: 0 additions & 21 deletions contracts/ResolutionManager/ResolutionManager.sol
Original file line number Diff line number Diff line change
Expand Up @@ -167,13 +167,6 @@ contract ResolutionManager is Initializable, ResolutionManagerBase, HasRole {
* @param resolutionId The id of the resolution to approve.
*/
function approveResolution(uint256 resolutionId) external virtual {
require(
_shareholderRegistry.isAtLeast(
_shareholderRegistry.MANAGING_BOARD_STATUS(),
_msgSender()
),
"Resolution: only managing board can approve"
);
_approveResolution(resolutionId);
}

Expand All @@ -182,13 +175,6 @@ contract ResolutionManager is Initializable, ResolutionManagerBase, HasRole {
* @param resolutionId The id of the resolution to reject.
*/
function rejectResolution(uint256 resolutionId) external virtual {
require(
_shareholderRegistry.isAtLeast(
_shareholderRegistry.MANAGING_BOARD_STATUS(),
_msgSender()
),
"Resolution: only managing board can reject"
);
_rejectResolution(resolutionId);
}

Expand All @@ -209,13 +195,6 @@ contract ResolutionManager is Initializable, ResolutionManagerBase, HasRole {
address[] memory executionTo,
bytes[] memory executionData
) external virtual {
require(
_shareholderRegistry.isAtLeast(
_shareholderRegistry.MANAGING_BOARD_STATUS(),
_msgSender()
),
"Resolution: only managing board can update"
);
_updateResolution(
resolutionId,
dataURI,
Expand Down

0 comments on commit d3aaa63

Please sign in to comment.