-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
fix: ensure edit nonce input only accepts numbers #8227
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. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8227 +/- ##
=======================================
Coverage 41.08% 41.09%
=======================================
Files 1245 1245
Lines 30330 30332 +2
Branches 2962 2962
=======================================
+ Hits 12462 12464 +2
Misses 17125 17125
Partials 743 743 ☔ View full report in Codecov by Sentry. |
E2E test started on Bitrise: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/93d86368-ea65-4a30-9c6f-fba937a9223d |
Editing nonce by manual input using numbers only (while having higher nonce stuck transactions) results in wrong high nonce: number.movEditing nonce using "+-" buttons works normal: plus_nonce.mov |
Same bug exists on main.mov |
Probably it is not a bug since we use computer keyboard to input characters on local build while on real Android device keyboard does not appear. Need to confirm the keyboard is unavailable on other mobile devices as well. If we don't get keyboard on mobile devices current issue is not a bug. |
E2E test started on Bitrise: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/cc368c88-b503-4fac-b53e-9a6538f84c3f |
E2E test started on Bitrise: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/9d4ace87-efb2-419d-a48b-00fefcd37565 |
I'm not able to use any other symbols but numbers in the 'edit nonce'. I'm able to edit nonce numbers on Confirmation screen using = / - buttons and numbers (cut-n-paste). But the transaction submits with next nonce instead of custom one. nonce.mov |
Editing nonce works as expected, low nonce value prompts the error message, high nonce transaction submits successfully. Screen_Recording_20240214_110346_MetaMask-QA.mp4 |
Quality Gate passedIssues Measures |
Description
This PR ensures the input field for edit nonce only accepts numbers.
Related issues
Fixes: #6857
Manual testing steps
Screenshots/Recordings
Before
After
numericNonce.webm
Pre-merge author checklist
Pre-merge reviewer checklist