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: signature redesign ui_customizations metric #24484
fix: signature redesign ui_customizations metric #24484
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. |
const isConfirmationRedesign = | ||
isConfirmationRedesignEnabled() && | ||
[...REDESIGN_APPROVAL_TYPES, ...REDESIGN_TRANSACTION_TYPES].find( | ||
(type) => type === method, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I removed the redesign transaction types check because it does not work. We will handle this with #24486
sorry I should start running yarn lint locally before requesting reviews 🤦🏼♀️ @cryptotavares @jpuri |
Builds ready [39f5866]
Page Load Metrics (2093 ± 724 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 #24484 +/- ##
===========================================
+ Coverage 67.47% 67.49% +0.01%
===========================================
Files 1288 1288
Lines 50154 50151 -3
Branches 13023 13018 -5
===========================================
+ Hits 33841 33845 +4
+ Misses 16313 16306 -7 ☔ View full report in Codecov by Sentry. |
Description
Related issues
Fixes: #24483
Manual testing steps
ENABLE_CONFIRMATION_REDESIGN=true
in .metamaskrcScreenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist