Closed NidhiKJha closed 1 week ago
@NidhiKJha you have 1 broken unit test
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.
I'm not sure if 'Select networks for each site' in Experimental settings should be ON by default? Seems like multichain doesn't work correctly? Sorry, if I'm missing something :).
https://github.com/MetaMask/metamask-extension/assets/104780023/a7056a1c-0876-46d5-9edc-978a3ac7b49e
Attention: Patch coverage is 30.00000%
with 14 lines
in your changes are missing coverage. Please review.
Project coverage is 67.45%. Comparing base (
ed850ee
) to head (ef8a63d
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Platform | Page | Metric | Min (ms) | Max (ms) | Average (ms) | StandardDeviation (ms) | MarginOfError (ms) |
---|---|---|---|---|---|---|---|
Chrome | Home | firstPaint | 70 | 183 | 106 | 28 | 14 |
domContentLoaded | 9 | 66 | 23 | 16 | 8 | ||
load | 56 | 3389 | 944 | 1324 | 636 | ||
domInteractive | 9 | 66 | 23 | 16 | 8 |
Platform | Page | Metric | Min (ms) | Max (ms) | Average (ms) | StandardDeviation (ms) | MarginOfError (ms) |
---|---|---|---|---|---|---|---|
Chrome | Home | firstPaint | 70 | 127 | 87 | 16 | 8 |
domContentLoaded | 9 | 44 | 17 | 9 | 4 | ||
load | 56 | 2626 | 659 | 1029 | 494 | ||
domInteractive | 9 | 44 | 17 | 9 | 4 |
This PR is to remove all the Multichain flags from code except the send and swap flow
Related issues
Fixes: https://github.com/MetaMask/MetaMask-planning/issues/2431
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist