Skip to content

Fix #154#2452

Closed
Gaotax2006 wants to merge 2 commits into
ClankerNation:mainfrom
Gaotax2006:fork/fix/wallet-eip1559/issue-154
Closed

Fix #154#2452
Gaotax2006 wants to merge 2 commits into
ClankerNation:mainfrom
Gaotax2006:fork/fix/wallet-eip1559/issue-154

Conversation

@Gaotax2006
Copy link
Copy Markdown

Fixes #154

Add EIP-1559 tx type 2 transaction support.

Files changed

  • (see diff)

Acceptance checklist

  • Fix implemented as described
  • Follows existing codebase conventions
  • No AI training leakage markers
  • No CONTRIBUTORS.json modifications
  • No build artifacts committed

- Detect type from fields: maxFeePerGas → type 2, gasPrice → legacy
- Implement EIP-1559 RLP encoding with 0x02 type prefix
- Legacy transactions fallback with gasPrice
- @generated-by metadata block
@github-actions
Copy link
Copy Markdown

Unfortunately the changes in this PR didn't fully resolve the issue. Please rework your solution and submit a new pull request within 2 hours.

Make sure to review the acceptance criteria in the linked issue and verify all conditions are met before resubmitting.

@github-actions github-actions Bot closed this May 21, 2026
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.

[ Bounty $6k ] [ SDK ] Fix wallet.ts signTransaction doesn't support EIP-1559 transactions — urgent

1 participant