refactor(web-devtools): remove-arbitrator-ruler-check

Author: tractorss

web-devtools/src/app/(main)/ruler/ChangeDeveloper.tsx

@@ -33,7 +33,7 @@ const StyledLabel = styled.label`
 `;
 
 const ChangeDeveloper: React.FC = () => {
-  const { arbitrable, currentDeveloper, refetchData, isRulerOfArbitrable } = useRulerContext();
+  const { arbitrable, currentDeveloper, refetchData } = useRulerContext();
   const [newDeveloper, setNewDeveloper] = useState("");
   const [isChanging, setIsChanging] = useState(false);
   const publicClient = usePublicClient();
@@ -46,7 +46,7 @@ const ChangeDeveloper: React.FC = () => {
     isError,
   } = useSimulateKlerosCoreRulerChangeRuler({
     query: {
-      enabled: !isUndefined(arbitrable) && !isUndefined(newDeveloper) && isRulerOfArbitrable && isAddress(newDeveloper),
+      enabled: !isUndefined(arbitrable) && !isUndefined(newDeveloper) && isAddress(newDeveloper),
     },
     args: [(arbitrable ?? "") as Address, newDeveloper as Address],
   });
@@ -77,7 +77,7 @@ const ChangeDeveloper: React.FC = () => {
         text="Update"
         onClick={handleClick}
         isLoading={isLoading || isChanging}
-        disabled={isError || isLoading || isChanging || isUndefined(arbitrable) || !isRulerOfArbitrable || !isValid}
+        disabled={!changeRulerConfig || isError || isLoading || isChanging || isUndefined(arbitrable) || !isValid}
       />
     </Container>
   );

web-devtools/src/app/(main)/ruler/ManualRuling.tsx

@@ -116,7 +116,7 @@ const ManualRuling: React.FC = () => {
         text="Rule"
         onClick={handleRuling}
         isLoading={isLoadingExecuteConfig || isSending}
-        disabled={isDisabled || isError || isSending}
+        disabled={isDisabled || isError || isSending || isLoadingExecuteConfig}
       />
     </Container>
   );

web-devtools/src/app/(main)/ruler/RulingModes.tsx

@@ -45,7 +45,7 @@ const AutomaticPresetInputsContainer = styled.div`
 `;
 
 const RulingModes: React.FC = () => {
-  const { arbitrable, arbitrableSettings, isRulerOfArbitrable } = useRulerContext();
+  const { arbitrable, arbitrableSettings } = useRulerContext();
   const [rulingMode, setRulingMode] = useState<RULING_MODE>();
   const [tie, setTie] = useState(false);
   const [overriden, setOverriden] = useState(false);
@@ -116,7 +116,7 @@ const RulingModes: React.FC = () => {
     useWriteKlerosCoreRulerChangeRulingModeToAutomaticRandom();
 
   const isDisabled = useMemo(() => {
-    if (!arbitrable || !isRulerOfArbitrable) return true;
+    if (!arbitrable) return true;
     switch (rulingMode) {
       case RULING_MODE.Manual:
         return (
@@ -155,7 +155,6 @@ const RulingModes: React.FC = () => {
     isLoadingManualConfig,
     isLoadingAutomaticRandomConfig,
     isLoadingAutomaticPresetConfig,
-    isRulerOfArbitrable,
     arbitrableSettings,
     tie,
     overriden,

web-devtools/src/context/RulerContext.tsx

@@ -21,7 +21,6 @@ interface IRulerContext {
   setArbitrable: (arbitrable: Address) => void;
   arbitrableSettings?: ArbitrableSettings;
   currentDeveloper?: Address;
-  isRulerOfArbitrable: boolean;
   refetchData: () => void;
 }
 const RulerContext = createContext<IRulerContext | undefined>(undefined);
@@ -62,8 +61,6 @@ const RulerContextProvider: React.FC<{ children: React.ReactNode }> = ({ childre
     refetchDeveloper();
   }, [refetchArbitrableSettings, refetchDeveloper]);
 
-  const isRulerOfArbitrable = useMemo(() => address === currentDeveloper, [address, currentDeveloper]);
-
   return (
     <RulerContext.Provider
       value={useMemo(
@@ -73,9 +70,8 @@ const RulerContextProvider: React.FC<{ children: React.ReactNode }> = ({ childre
           arbitrableSettings,
           currentDeveloper,
           refetchData,
-          isRulerOfArbitrable,
         }),
-        [arbitrable, setArbitrable, arbitrableSettings, currentDeveloper, refetchData, isRulerOfArbitrable]
+        [arbitrable, setArbitrable, arbitrableSettings, currentDeveloper, refetchData]
       )}
     >
       {children}