fix: governor mod token validation

[juliopavila]

Bug Report

https://discord.com/channels/881881751369175040/1032974561735606322/1268312094386950286

Implementation

In the packages/app/src/views/AddModule/wizards/OzGovernor/service/tokenValidation.ts file, the following changes were made to fix the bug:

1. Replaced tokenContract.delegates(RANDOM_VALID_ADDRESS) with tokenContract.delegates.staticCall(RANDOM_VALID_ADDRESS).
2. Removed the lines:

   // @ts-ignore
   tokenContract.functions['delegateBySig'].name

3. Added the line:

   tokenContract.delegateBySig.name

These changes ensure the application correctly identifies and calls the delegateBySig function on the contract, addressing the issues caused by the proxy contract used by the Arbitrum Foundation.

Additional Context

The error reported was:

Error: missing revert data (action="call", data=null, reason=null, transaction={ "data": "0x9ab24eb0000000000000000000000000d028d504316fec029cfa36bdc3a8f053f6e5a6e4", "to": "0xFF970A61A04b1cA14834A43f5dE4533eBDDB5CC8" }, invocation=null, revert=null, code=CALL_EXCEPTION, version=6.13.1)
    at makeError (errors.ts:694:21)
    at getBuiltinCallException (abi-coder.ts:118:12)
    at _AbiCoder.getBuiltinCallException (abi-coder.ts:235:16)
    at BrowserProvider.getRpcError (provider-jsonrpc.ts:989:32)
    at BrowserProvider.getRpcError (provider-browser.ts:122:22)
    at provider-jsonrpc.ts:563:45

This error occurred while testing with the Arbitrum Foundation's address 0x912CE59144191C1204E64559FE8253a0e49E6548. The application was not seeing the delegateBySig() function on the contract, likely due to the proxy contract being used.

---

Commit Changes

This change ensures that the function calls are correctly handled and improves compatibility with contracts deployed on all networks, specifically addressing the issues with proxy contracts.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
zodiac-safe-app-ui	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Aug 1, 2024 8:05pm

[auryn-macmillan]

lgtm. Thanks @juliopavila!

[auryn-macmillan]

@juliopavila, would you mind also bumping the package version so we can publish a new release immediately?

[auryn-macmillan]

lgtm