Skip to content

feat: Add support for type 4 transaction#521

Merged
dan437 merged 4 commits intomainfrom
7702-tx-support
Jul 15, 2025
Merged

feat: Add support for type 4 transaction#521
dan437 merged 4 commits intomainfrom
7702-tx-support

Conversation

@dan437
Copy link
Copy Markdown
Collaborator

@dan437 dan437 commented Jul 15, 2025

Description

Adds support for type 4 transaction, refactor txHash parsing.

Testing

  • Include the STX controller with this change in the extension, e.g. via yalc

Test for type 4 transaction:

  • Have BNB chain selected in the extension
  • Turn off this Advanced setting
image
  • Go on jumper.exchange
  • Connect MM wallet with an account that is not yet upgraded to a Smart Account
  • Do a swap e.g. from BUSD -> BNB
  • Once submitted, UI will show if you want to upgrade to Smart Account. The transaction will work

Regular tests:

  • Make sure that smart transactions still work in the Send, Swap and dapp flows

dan437 added 2 commits July 14, 2025 14:00
Signed-off-by: dan437 <80175477+dan437@users.noreply.github.com>
Signed-off-by: dan437 <80175477+dan437@users.noreply.github.com>
@dan437 dan437 requested a review from a team July 15, 2025 14:44
cursor[bot]

This comment was marked as outdated.

@socket-security
Copy link
Copy Markdown

socket-security Bot commented Jul 15, 2025

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updated@​ethersproject/​keccak256@​5.7.0 ⏵ 5.8.01001006977100

View full report

klejeune
klejeune previously approved these changes Jul 15, 2025
dan437 added 2 commits July 15, 2025 17:30
Signed-off-by: dan437 <80175477+dan437@users.noreply.github.com>

Update deps

Signed-off-by: dan437 <80175477+dan437@users.noreply.github.com>

Update

Signed-off-by: dan437 <80175477+dan437@users.noreply.github.com>
@dan437 dan437 merged commit a7af380 into main Jul 15, 2025
18 checks passed
@dan437 dan437 deleted the 7702-tx-support branch July 15, 2025 15:36
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.

2 participants