fix: safe tx gas price should be zero - Githubissues

ApeWorX / ape-safe

Safe (Wallet) account plugin for the Ape Framework

https://www.apeworx.io/

Apache License 2.0

14 stars 8 forks source link

fix: safe tx gas price should be zero #52

Closed banteg closed 4 months ago

banteg commented 5 months ago

What I did

gas price has a different meaning in safe transactions. it is used to pay the relayer which is not what we want.

https://github.com/safe-global/safe-smart-account/blob/a9e3385bb38c29d45b3901ff7180b59fcee86ac9/contracts/Safe.sol#L180-L182

fixes inability to execute submitted transactions.

How I did it

force gas_price to zero

How to verify it

Checklist

[ ] Passes all linting checks (pre-commit and CI jobs)
[ ] New test cases have been added and are passing
[ ] Documentation has been updated
[ ] PR title follows Conventional Commit standard (will be automatically included in the changelog)