-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
feat: Change copy from estimated to network fee #26859
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. |
08f6061
to
5c92007
Compare
5c92007
to
0ae5a93
Compare
Quality Gate passedIssues Measures |
Builds ready [0ae5a93]
Page Load Metrics (1876 ± 103 ms)
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #26859 +/- ##
===========================================
- Coverage 70.13% 70.13% -0.00%
===========================================
Files 1424 1424
Lines 49571 49571
Branches 13867 13867
===========================================
- Hits 34765 34764 -1
- Misses 14806 14807 +1 ☔ View full report in Codecov by Sentry. |
Description
Update copy and corresponding tests.
Related issues
Fixes: https://github.com/MetaMask/MetaMask-planning/issues/3063
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist