Skip to content
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

Cherry pick 7.42.0 540927448b fix #13802

Closed
wants to merge 3 commits into from

Conversation

salimtb
Copy link
Contributor

@salimtb salimtb commented Mar 3, 2025

cherry-pick #13740

metamaskbot and others added 3 commits February 27, 2025 23:07
PR to fix balance display in signature confirmation component.
It also fixes the balance display when the network is being switched
within a dapp.

Fixes: #13730
Fixes: #13727

Description of manual testing steps are included in reported issues
1. Go to this page...
2.
3.

<!-- If applicable, add screenshots and/or recordings to visualize the
before and after of your change. -->

<!-- [screenshots/recordings] -->

<!-- [screenshots/recordings] -->
[Screen Recording 2025-02-26 at
19.09.10](https://github.com/user-attachments/assets/a4fd0edc-e81d-4284-a2b9-44dbe5b90a49)

https://github.com/user-attachments/assets/13e39779-588d-43ba-a41b-dea9e1ce5b48

- [ ] I’ve followed [MetaMask Contributor
Docs](https://github.com/MetaMask/contributor-docs) and [MetaMask Mobile
Coding
Standards](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/CODING_GUIDELINES.md).
- [ ] 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
- [ ] I’ve applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-mobile/blob/main/.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: Prithpal Sooriya <[email protected]>
Co-authored-by: salimtb <[email protected]>
Copy link
Contributor

github-actions bot commented Mar 3, 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 team-assets INVALID-PR-TEMPLATE PR's body doesn't match template labels Mar 3, 2025
@salimtb salimtb closed this Mar 3, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Mar 3, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
INVALID-PR-TEMPLATE PR's body doesn't match template team-assets
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants