From c56e802ed08020e8df0ab3cf3d938af11d8cfc50 Mon Sep 17 00:00:00 2001
From: Rob Green <rbgrn@rbgrn.net>
Date: Wed, 25 Oct 2023 09:09:51 -0700
Subject: [PATCH] Made scope change events use plain string instead of index
 (#33)

---
 src/PatchworkProtocol.sol | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/PatchworkProtocol.sol b/src/PatchworkProtocol.sol
index 7933228..d851290 100644
--- a/src/PatchworkProtocol.sol
+++ b/src/PatchworkProtocol.sol
@@ -282,7 +282,7 @@ 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
@@ -290,7 +290,7 @@ contract PatchworkProtocol {
     @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
@@ -298,7 +298,7 @@ contract PatchworkProtocol {
     @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
@@ -306,7 +306,7 @@ contract PatchworkProtocol {
     @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
@@ -316,7 +316,7 @@ 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
@@ -324,7 +324,7 @@ contract PatchworkProtocol {
     @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
@@ -332,7 +332,7 @@ contract PatchworkProtocol {
     @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