-
Notifications
You must be signed in to change notification settings - Fork 5.2k
fix: missing deadline in swaps stx status screen (#25779) #26544
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
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. |
4ffd6a3
to
98d497e
Compare
<!-- Please submit this PR as a draft initially. Do not mark it as "Ready for review" until the template has been completely filled out, and PR status checks have passed at least once. --> <!-- Write a short description of the changes included in this pull request, also include relevant motivation and context. Have in mind the following questions: 1. What is the reason for the change? 2. What is the improvement/solution? --> This PR fixes an issue where the STX status screen for a swap was showing a 0:00 for the timer. [](https://codespaces.new/MetaMask/metamask-extension/pull/25779?quickstart=1) Related to: #25063 1. Make sure Smart Transactions is on (Settings > Advanced) 2. Do a Swap 3. Observe that timer is not 0:00 and is a reasonable number <!-- If applicable, add screenshots and/or recordings to visualize the before and after of your change. --> <!-- [screenshots/recordings] --> https://github.com/MetaMask/metamask-extension/assets/139582705/26fe6167-614f-4771-b35b-10803bc23fc0 <!-- [screenshots/recordings] --> https://github.com/MetaMask/metamask-extension/assets/139582705/d92b933d-1011-48b4-bf04-344f275d35db - [x] I've followed [MetaMask Contributor Docs](https://github.com/MetaMask/contributor-docs) and [MetaMask Extension Coding Standards](https://github.com/MetaMask/metamask-extension/blob/develop/.github/guidelines/CODING_GUIDELINES.md). - [x] I've completed the PR template to the best of my ability - [ ] I’ve included tests if applicable - [ ] I’ve documented my code using [JSDoc](https://jsdoc.app/) format if applicable - [x] I’ve applied the right labels on the PR (see [labeling guidelines](https://github.com/MetaMask/metamask-extension/blob/develop/.github/guidelines/LABELING_GUIDELINES.md)). Not required for external contributors. - [ ] 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. --------- Co-authored-by: Marta Poling <[email protected]>
98d497e
to
d7e95af
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## Version-v12.1.0 #26544 +/- ##
===================================================
- Coverage 69.64% 69.64% -0.00%
===================================================
Files 1363 1363
Lines 48452 48454 +2
Branches 13363 13365 +2
===================================================
Hits 33743 33743
- Misses 14709 14711 +2 ☔ View full report in Codecov by Sentry. |
Builds ready [d7e95af]
Page Load Metrics (450 ± 357 ms)
|
Description
This PR fixes an issue where the STX status screen for a swap was showing a 0:00 for the timer. It is a cherry-pick of this PR.
Related issues
Related to: #25063
Manual testing steps
Screenshots/Recordings
Before
before.mov
After
Screen.Recording.2024-08-20.at.9.39.38.AM.mov
Pre-merge author checklist
Pre-merge reviewer checklist