Skip to content

fix(Tron): transaction syncing failures#28549

Draft
ulissesferreira wants to merge 2 commits intomainfrom
ulissesferreira/chore/tron-wallet-snap-preview-289
Draft

fix(Tron): transaction syncing failures#28549
ulissesferreira wants to merge 2 commits intomainfrom
ulissesferreira/chore/tron-wallet-snap-preview-289

Conversation

@ulissesferreira
Copy link
Copy Markdown
Contributor

@ulissesferreira ulissesferreira commented Apr 8, 2026

Description

This PR overrides @metamask/tron-wallet-snap to use the latest preview package published from MetaMask/snap-tron-wallet#289: @metamask-previews/tron-wallet-snap@1.25.1-preview-51f983e.

The goal is to pull the upstream Tron transaction syncing fixes into Mobile without waiting for a stable snap release. The change is limited to Yarn resolution and lockfile updates so installs consistently resolve to the preview artifact.

Changelog

CHANGELOG entry: Updated the Tron wallet snap to include the latest transaction syncing fixes.

Related issues

Fixes:

Manual testing steps

Feature: Tron wallet snap preview override

  Scenario: user views Tron activity with the preview snap build
    Given I am running MetaMask Mobile from this branch
    And I have a wallet with a Tron account that has recent activity

    When user opens the Tron account
    And user views the activity list
    Then Tron transactions should sync successfully
    And the activity list should render without transaction sync failures

  Scenario: user loads a Tron account with TRC10 activity
    Given I am running MetaMask Mobile from this branch
    And I have a wallet with a Tron account that includes TRC10 token activity

    When user refreshes the Tron account activity
    Then the activity sync should complete successfully
    And TRC10 activity should load without malformed token identifier errors

Screenshots/Recordings

Before

N/A

After

N/A

Pre-merge author checklist

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

@ulissesferreira ulissesferreira self-assigned this Apr 8, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 8, 2026

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@metamaskbot metamaskbot added the team-networks Networks team label Apr 8, 2026
@socket-security
Copy link
Copy Markdown

socket-security bot commented Apr 8, 2026

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

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatednpm/​@​metamask/​tron-wallet-snap@​1.25.0 ⏵ 1.25.1-preview-51f983e100 +1100100 +5100 +2100

View full report

@ulissesferreira ulissesferreira changed the title build(deps): use tron wallet snap preview build build(deps): preview build to test https://github.com/MetaMask/snap-tron-wallet/pull/289 Apr 8, 2026
@ulissesferreira ulissesferreira changed the title build(deps): preview build to test https://github.com/MetaMask/snap-tron-wallet/pull/289 build(deps): "fix: transaction syncing failures" on https://github.com/MetaMask/snap-tron-wallet/pull/289 Apr 8, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 8, 2026

🔍 Smart E2E Test Selection

⏭️ Smart E2E selection skipped - draft PR

All E2E tests pre-selected.

View GitHub Actions results

@ulissesferreira ulissesferreira changed the title build(deps): "fix: transaction syncing failures" on https://github.com/MetaMask/snap-tron-wallet/pull/289 fix(Tron): transaction syncing failures Apr 8, 2026
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 8, 2026

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 8, 2026

E2E Fixture Validation — Schema is up to date
15 value mismatches detected (expected — fixture represents an existing user).
View details

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants