search

omgnetwork / optimism

Monorepo implementing the Optimistic Ethereum protocol
https://optimism.io
MIT License
16 stars 6 forks source link

Extra approval when staking OMG tokens on l2 #511

Closed Ralph89 closed 3 years ago

Ralph89 commented 3 years ago

So steps to reproduce:

1: Approve the amount to stake on l2 2: Cancel the tx to stake (I did not cancel though, but I was doing something afk and went to screensaver, when I came back there was no tx available to stake, I needed to approve the amount I wanted to stake once more. resulting in 2 approve messages. I am quite sure if I cancelled the stake tx that popup after wards I would need to approve the amount once more

An idea is to check if the amount is approved, instead of asking to approve the amount everytime after declining / reloading a page.

CAPtheorem commented 3 years ago

L1 staking for OMG will be fixed today. (The problem is the same as with the OMG bridging).

CAPtheorem commented 3 years ago

Fixing in https://github.com/omgnetwork/optimism/pull/515 - will be live within a few hours.