Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Resolve WalletConnect Issues #74

Open
earth2travis opened this issue Jun 2, 2023 · 6 comments
Open

Resolve WalletConnect Issues #74

earth2travis opened this issue Jun 2, 2023 · 6 comments

Comments

@earth2travis
Copy link
Member

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
Copy link
Member Author

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

Image

@earth2travis earth2travis changed the title WalletConnect issues Resolve WalletConnect Issues Jun 2, 2023
@earth2travis
Copy link
Member Author

@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
Copy link
Member Author

Image

@earth2travis
Copy link
Member Author

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
Copy link
Member Author

@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
Copy link
Member Author

earth2travis commented Aug 31, 2023

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

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 });
};

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant