Resolve WalletConnect Issues

earth2travis commented 1 year ago

https://discord.com/channels/709210493549674598/1082758136093478932/1111644513841184838

Gather information from Russell on Wednesday. Record video replicating issue. Assuming this is related to WalletConnect and Polygon.

earth2travis commented 1 year ago

When trying to add a custom safe app from the Gnosis Safe:

earth2travis commented 1 year ago

@skuhlmann

WalletConnect issue is related to Connect Wallet not actually disconnecting from wallet. If you Disconnect and hit refresh it automatically connects to previously connected wallet. This is not allowing him to switch to another wallet. Can manually disconnect from site in MetaMask browser extension but cannot disconnect from an individual site in MetaMask mobile. He is forced to use new incognito windows.

earth2travis commented 1 year ago

The Connect Wallet caching issue is fairly common in the space. Another project I was working on is having similiar issues with a completely different wallet package (Web3 Onboard from Blocknative). We could try to update the Connect Wallet package and use something like RainbowKit but that requires a signifiant update to a package in the core product. I can see if we can get you a quote if this is something you want to pursue.

earth2travis commented 1 year ago

@skuhlmann remembered that we forked our repo into the SporkDAOOfficial organization

https://github.com/SporkDAOOfficial/bufficorn-dao

Will need to make sure to get any changes merged into that once the WalletConnect issues are resolved

earth2travis commented 1 year ago

TransactionNotFoundError

While using the Gnosis Safe app to interact with the DAO we are seeing transaction errors in the UI and console.

Errors

When creating a proposal

TransactionNotFoundError

The transactions are actually going through but they are

This is a link to the failed transaction: https://polygonscan.com/tx/0x4a10630d1df8bab9b564026496a2e0e72862f5c6f82805383ffb4b1b0382b048
But the proposal actually went through: https://admin.daohaus.club/#/molochV3/0x89/0x3102fdcaaf1c08116142caaa8f32b647ebf34ba3/proposals/19

When voting on a proposal

safe-ui

daohaus-ui

Diagnosis

It appears Viem is too fast for the RPC. Polygon RPCs have historically been an issue but I am also able to replicate the issue on Gnosis Chain.

There is currently a related open issue on Github bug: TransactionReceiptNotFoundError: Transaction receipt with hash #1056

We may need to find a way to handle the error better but there is little we can do here. It might be possible to replace Rivet with Alchemy. Unsure if this will work but may be worth trying. How to Create a Private RPC Endpoint

We also may trying increasing the retryCount in createTransport

export const createTransport = ({
  chainId,
  rpcs = HAUS_RPC,
  retryCount = 5,
}: {
  chainId: ValidNetwork;
  rpcs: Keychain;
  retryCount?: number;
}): HttpTransport => {
  const rpc = rpcs[chainId];
  if (!rpc) return http();
  return http(rpc, { retryCount });
};

0xBootleggers / bufficorn