metamask-extension icon indicating copy to clipboard operation
metamask-extension copied to clipboard

Modify smart-swap and swap-eth scenarios based on current feature change

Open benjisclowder opened this issue 1 year ago • 4 comments

Smart Swap has changed from a swap setting screen toggle to Smart Transactions in Settings toggle. This PR reflects that change.

benjisclowder avatar May 22 '24 14:05 benjisclowder

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.

github-actions[bot] avatar May 22 '24 14:05 github-actions[bot]

Builds ready [659a94e]
Page Load Metrics (1599 ± 613 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint631781113718
domContentLoaded95516105
load48320315991278613
domInteractive95516105
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: 0 Bytes (0.00%)
  • common: 0 Bytes (0.00%)

metamaskbot avatar May 22 '24 14:05 metamaskbot

LGTM!

chloeYue avatar May 23 '24 07:05 chloeYue

@seaona @chloeYue. There was another scenario that needed changes - the ENS one, to reflect the fact that ENS name resolution only works on Ethereum Mainnet. Potentially related to: Resolve ENS Names on L2 chains. I have updated this since Customer Success always mentions this scenario not working with Sepolia as indicated in the test steps. Should this change in the future, we will update the scenario again once it's done. So this needs also to be reviewed. Thank you

benjisclowder avatar May 23 '24 12:05 benjisclowder

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 65.85%. Comparing base (50fdee3) to head (412be75).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop   #24711   +/-   ##
========================================
  Coverage    65.85%   65.85%           
========================================
  Files         1363     1363           
  Lines        54254    54254           
  Branches     14114    14114           
========================================
  Hits         35725    35725           
  Misses       18529    18529           

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar May 30 '24 20:05 codecov[bot]

Builds ready [412be75]
Page Load Metrics (1371 ± 572 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint721611143014
domContentLoaded96419167
load60316713711191572
domInteractive96419167
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: 0 Bytes (0.00%)
  • common: 0 Bytes (0.00%)

metamaskbot avatar May 30 '24 20:05 metamaskbot