Merge pull request #62 from gnosis/add-events

Add events required by ERC-5005
gnosisguild · Jul 13, 2022 · d3d0e49 · d3d0e49
2 parents 1a2c976 + f26eac8
commit d3d0e49
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/contracts/core/Modifier.sol b/contracts/core/Modifier.sol
@@ -11,9 +11,6 @@ abstract contract Modifier is Module, IAvatar {
     /// Mapping of modules.
     mapping(address => address) internal modules;
 
-    event EnabledModule(address module);
-    event DisabledModule(address module);
-
     /// `sender` is not an authorized module.
     /// @param sender The address of the sender.
     error NotAuthorized(address sender);

diff --git a/contracts/interfaces/IAvatar.sol b/contracts/interfaces/IAvatar.sol
@@ -6,6 +6,11 @@ pragma solidity >=0.7.0 <0.9.0;
 import "@gnosis.pm/safe-contracts/contracts/common/Enum.sol";
 
 interface IAvatar {
+    event EnabledModule(address module);
+    event DisabledModule(address module);
+    event ExecutionFromModuleSuccess(address indexed module);
+    event ExecutionFromModuleFailure(address indexed module);
+
     /// @dev Enables a module on the avatar.
     /// @notice Can only be called by the avatar.
     /// @notice Modules should be stored as a linked list.