Skip to content

Conversation

@PatrykLucka
Copy link
Contributor

@PatrykLucka PatrykLucka commented Nov 7, 2025

Description

This PR fixes styling for "unconfirmed" transaction status

Changelog

CHANGELOG entry: Fixed styling for "unconfirmed" transaction status

Related issues

Fixes: #21574
https://consensyssoftware.atlassian.net/browse/TMCU-158

Manual testing steps

Feature: Send BTC

  Scenario: user sends BTC
    Given BTC is enabled

    When user initiates BTC transaction and navigates to Activity tab
    Then unconfirmed transaction with yellow/orange "Pending" status should appear

Screenshots/Recordings

Before

Screenshot 2025-11-07 at 13 53 14

After

Screenshot 2025-11-07 at 13 52 51

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.

Note

Maps Unconfirmed statuses to pending and enables overriding styles on status text components; updates i18n and snapshots.

  • UI:
    • app/components/Base/StatusText.js:
      • ConfirmedText, PendingText, FailedText now accept optional style prop merged with base styles.
      • Treat "Unconfirmed"/"unconfirmed" as pending in StatusText switch.
  • i18n:
    • locales/languages/en.json: add "transaction.unconfirmed": "Pending".
  • Tests:
    • Update TransactionDetails snapshots to reflect array-based style merging.

Written by Cursor Bugbot for commit fa44247. This will update automatically on new commits. Configure here.

@PatrykLucka PatrykLucka self-assigned this Nov 7, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Nov 7, 2025

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-mobile-ux Mobile UX team label Nov 7, 2025
@github-actions github-actions bot added the size-M label Nov 7, 2025
@github-actions github-actions bot added size-S and removed size-M labels Nov 7, 2025
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 7, 2025

@PatrykLucka PatrykLucka marked this pull request as ready for review November 7, 2025 14:33
@PatrykLucka PatrykLucka enabled auto-merge November 7, 2025 14:34
auto-merge was automatically disabled November 10, 2025 17:30

Pull Request is not mergeable

@aganglada aganglada changed the title fix: unconfirmed status styling fix: cp-7.59.0 unconfirmed status styling Nov 11, 2025
@PatrykLucka PatrykLucka added this pull request to the merge queue Nov 12, 2025
Merged via the queue into main with commit 50ca0df Nov 12, 2025
89 checks passed
@PatrykLucka PatrykLucka deleted the TMCU-158-bug-transaction-pending-status-missing-colour-and-translation branch November 12, 2025 08:43
@github-actions github-actions bot locked and limited conversation to collaborators Nov 12, 2025
@metamaskbot metamaskbot added the release-7.60.0 Issue or pull request that will be included in release 7.60.0 label Nov 12, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release-7.60.0 Issue or pull request that will be included in release 7.60.0 size-S team-mobile-ux Mobile UX team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Transaction pending status missing colour and translation

4 participants