MetaMask / metamask-extension

:globe_with_meridians: :electric_plug: The MetaMask browser extension enables browsing Ethereum blockchain enabled websites
https://metamask.io
Other
12.02k stars 4.91k forks source link

[Bug]: Native Send - Metrics - When I have the redesign toggle disabled I still see Send ETH transactions with `redesigned_confirmation` in Metrics Events #28322

Open seaona opened 1 day ago

seaona commented 1 day ago

Describe the bug

When I have the redesign toggle disabled and I send a native token (ie Send ETH) I still see transactions with redesigned_confirmation in Metrics Events, for transactions with the old designs. This happens for both when the Send is initiated from the wallet or from a dapp.

Expected behavior

We should not see the redesigned_confirmation in the ui_customizations for metrics events, when the redesign is off

Screenshots/Recordings

https://github.com/user-attachments/assets/188b7284-2ecc-4d69-a11d-314b37a561d2

Steps to reproduce

  1. Disable transactions redesign
  2. Enable metrics
  3. Trigger a send ETH transaction
  4. See the old design, but the metric event has the redesigned_confirmation

Error messages or log output

No response

Detection stage

During release testing

Version

12.7.0

Build type

None

Browser

Chrome

Operating system

Linux

Hardware wallet

No response

Additional context

No response

Severity

No response

seaona commented 1 day ago

:information_source: : @bschorchit adding the release-blocker tag, as all regressions are considered release-blockers by default. Feel free to remove the label if you want to re-assess the priority