fix(gas): use estimateGas for metaAccount transactions (Closes #219)#298
fix(gas): use estimateGas for metaAccount transactions (Closes #219)#298yanyishuai wants to merge 1 commit into
Conversation
|
@austintgriffith Follow-up on #219 estimateGas metaAccount support — CI green, zero competing open PRs. Ready for review/merge when convenient. Wallet: |
|
External bounty ready for maintainer review — acceptance criteria documented in PR body. Happy to address feedback immediately. |
|
@austintgriffith Follow-up on #219 — estimateGas metaAccount fix remains the only open PR, CI green, zero competing PRs. Merge-ready when you have bandwidth. Wallet: |
4 similar comments
|
@austintgriffith Follow-up on #219 — estimateGas metaAccount fix remains the only open PR, CI green, zero competing PRs. Merge-ready when you have bandwidth. Wallet: |
|
@austintgriffith Follow-up on #219 — estimateGas metaAccount fix remains the only open PR, CI green, zero competing PRs. Merge-ready when you have bandwidth. Wallet: |
|
@austintgriffith Follow-up on #219 — estimateGas metaAccount fix remains the only open PR, CI green, zero competing PRs. Merge-ready when you have bandwidth. Wallet: |
|
@austintgriffith Follow-up on #219 — estimateGas metaAccount fix remains the only open PR, CI green, zero competing PRs. Merge-ready when you have bandwidth. Wallet: |
1 participant
Summary
Adds
estimateTxGashelper with a 20% buffer and replaces hard-coded gas limits on metaAccountsignTransactionpaths in App, Exchange, and WithdrawFromPrivate.Closes #219
Wallet:
Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE