Skip to content

Closes #247: Improve requiredTxGas#266

Merged
rmeissner merged 1 commit intodevelopmentfrom
feature/issue_247_improve_required_tx_gas
Mar 2, 2021
Merged

Closes #247: Improve requiredTxGas#266
rmeissner merged 1 commit intodevelopmentfrom
feature/issue_247_improve_required_tx_gas

Conversation

@rmeissner
Copy link
Copy Markdown
Member

Closes #247:

  • Remove authorized from requiredTxGas
    • This allows to create an intermediate contract that converts the revert message into a normal return value

@rmeissner rmeissner force-pushed the feature/issue_247_improve_required_tx_gas branch from 4d4d512 to f480563 Compare March 1, 2021 22:02
@rmeissner rmeissner marked this pull request as ready for review March 1, 2021 22:02
Comment thread test/core/GnosisSafe.Estimation.spec.ts
Comment thread test/core/GnosisSafe.Estimation.spec.ts
@rmeissner rmeissner merged commit 1787935 into development Mar 2, 2021
@rmeissner rmeissner deleted the feature/issue_247_improve_required_tx_gas branch March 2, 2021 10:49
Uxio0 pushed a commit that referenced this pull request May 6, 2021
Saw-mon-and-Natalie pushed a commit to Saw-mon-and-Natalie/safe-contracts that referenced this pull request Nov 1, 2023
fdarian pushed a commit to fdarian/safe-contracts that referenced this pull request Jan 14, 2024
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

Successfully merging this pull request may close these issues.

Remove authorized from requiredTxGas

2 participants