Skip to content

Commit

Permalink
Made scope change events use plain string instead of index (#33)
Browse files Browse the repository at this point in the history
  • Loading branch information
robdoesstuff authored Oct 25, 2023
1 parent 92ba642 commit c56e802
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions src/PatchworkProtocol.sol
Original file line number Diff line number Diff line change
Expand Up @@ -282,31 +282,31 @@ contract PatchworkProtocol {
@param scopeName The name of the claimed scope
@param owner The owner of the scope
*/
event ScopeClaim(string indexed scopeName, address indexed owner);
event ScopeClaim(string scopeName, address indexed owner);

/**
@notice Emitted when a scope is transferred
@param scopeName The name of the transferred scope
@param from The address transferring the scope
@param to The recipient of the scope
*/
event ScopeTransfer(string indexed scopeName, address indexed from, address indexed to);
event ScopeTransfer(string scopeName, address indexed from, address indexed to);

/**
@notice Emitted when a scope has an operator added
@param scopeName The name of the scope
@param actor The address responsible for the action
@param operator The new operator's address
*/
event ScopeAddOperator(string indexed scopeName, address indexed actor, address indexed operator);
event ScopeAddOperator(string scopeName, address indexed actor, address indexed operator);

/**
@notice Emitted when a scope has an operator removed
@param scopeName The name of the scope
@param actor The address responsible for the action
@param operator The operator's address being removed
*/
event ScopeRemoveOperator(string indexed scopeName, address indexed actor, address indexed operator);
event ScopeRemoveOperator(string scopeName, address indexed actor, address indexed operator);

/**
@notice Emitted when a scope's rules are changed
Expand All @@ -316,23 +316,23 @@ contract PatchworkProtocol {
@param allowUserAssign Indicates whether user assignments are allowed
@param requireWhitelist Indicates whether a whitelist is required
*/
event ScopeRuleChange(string indexed scopeName, address indexed actor, bool allowUserPatch, bool allowUserAssign, bool requireWhitelist);
event ScopeRuleChange(string scopeName, address indexed actor, bool allowUserPatch, bool allowUserAssign, bool requireWhitelist);

/**
@notice Emitted when a scope has an address added to the whitelist
@param scopeName The name of the scope
@param actor The address responsible for the action
@param addr The address being added to the whitelist
*/
event ScopeWhitelistAdd(string indexed scopeName, address indexed actor, address indexed addr);
event ScopeWhitelistAdd(string scopeName, address indexed actor, address indexed addr);

/**
@notice Emitted when a scope has an address removed from the whitelist
@param scopeName The name of the scope
@param actor The address responsible for the action
@param addr The address being removed from the whitelist
*/
event ScopeWhitelistRemove(string indexed scopeName, address indexed actor, address indexed addr);
event ScopeWhitelistRemove(string scopeName, address indexed actor, address indexed addr);

/**
@notice Claim a scope
Expand Down

0 comments on commit c56e802

Please sign in to comment.