5afe / safe-react

Deprecated! New repo – https://github.com/safe-global/web-core
MIT License
332 stars 363 forks source link

Tx in BSC cannot be executed #3962

Closed francovenica closed 2 years ago

francovenica commented 2 years ago

Description

The submit button gets stuck in a pending status during estimation. No errors in the console or the network tab. This is only for the "execution" step, creating a tx with a safe 2/2 works, is execution that is not possible Trying to create a tx in a 1/1 safe has the same issue, the submit button gets stuck in a pending state

Tried the same safe and same tx in prod and there this issue doesn't happen

Environment

Steps to reproduce

  1. Open a safe 1/1 in bsc
  2. Try to add a new owner
  3. Try to sign and execute the tx

Expected result

The review form shows the submit button enabled after estimations

Obtained result

Even after estimations are done the submit button remains in a pending status

Screenshots

image image

francovenica commented 2 years ago

I'm having the same issue in arbitrum.

I don't have this issue in the GNO chain, polygon, or rinkeby

katspaugh commented 2 years ago

BSC seems to work for me. Arbitrum's public RPC is having issues. Transactions still work tho.

francovenica commented 2 years ago

I tried 2 more times during the weekend, several hours apart, and I still have the same issue.

I left a tx ready to be executed here https://pr3943--safereact.review-safe.gnosisdev.com/app/bnb:0x91e11585c114129f3Ec940Aa648A4ac13668d0c2/transactions/queue

You don't need funds in your wallet, just try to execute and the submit button will be stuck in a pending status

Same here in arbitrum, same situation: https://pr3943--safereact.review-safe.gnosisdev.com/app/arb1:0x4a6487a6634E0cD80ba7D15EcAE7F848FE8DF20B/transactions/queue

katspaugh commented 2 years ago

For me it gets stuck in the Estimating status as well.

Both safeTxGas and eth gas estimation return a correct response. eth_gasPrice as well. However, the nonce is displayed as 0 for some reason.

Screenshot 2022-06-13 at 09 28 36