From b2e46e654224d4ad1a2ff3ca9ffa6a12bd33f691 Mon Sep 17 00:00:00 2001
From: Sylva Elendu <iamsylvaelendu@gmail.com>
Date: Wed, 20 Sep 2023 12:49:42 +0100
Subject: [PATCH] unneeded code

---
 app/_locales/en/messages.json                 |  3 ++-
 ...ecurity-provider-banner-alert.test.js.snap |  2 +-
 .../security-provider-banner-alert.js         | 10 ++--------
 .../transaction-alerts/transaction-alerts.js  | 19 +------------------
 4 files changed, 6 insertions(+), 28 deletions(-)

diff --git a/app/_locales/en/messages.json b/app/_locales/en/messages.json
index 302468a59a8b..6bb49c2e7d2c 100644
--- a/app/_locales/en/messages.json
+++ b/app/_locales/en/messages.json
@@ -4142,7 +4142,8 @@
     "message": "Some networks may pose security and/or privacy risks. Understand the risks before adding & using a network."
   },
   "somethingDoesntLookRight": {
-    "message": "Something doesn't look right? $1"
+    "message": "Something doesn't look right? $1",
+    "description": "A false positive message for users to contact support. $1 is a link to the support page."
   },
   "somethingIsWrong": {
     "message": "Something's gone wrong. Try reloading the page."
diff --git a/ui/components/app/security-provider-banner-alert/__snapshots__/security-provider-banner-alert.test.js.snap b/ui/components/app/security-provider-banner-alert/__snapshots__/security-provider-banner-alert.test.js.snap
index ad92f2bc8eaf..ff549ac5dc51 100644
--- a/ui/components/app/security-provider-banner-alert/__snapshots__/security-provider-banner-alert.test.js.snap
+++ b/ui/components/app/security-provider-banner-alert/__snapshots__/security-provider-banner-alert.test.js.snap
@@ -58,7 +58,7 @@ exports[`Security Provider Banner Alert should match snapshot 1`] = `
         </details>
       </div>
       <p
-        class="mm-box mm-text mm-text--body-sm mm-box--margin-top-3 mm-box--display-flex mm-box--align-items-center mm-box--color-text-alternative"
+        class="mm-box mm-text mm-text--body-md mm-box--margin-top-3 mm-box--display-flex mm-box--color-text-default"
       >
         [somethingDoesntLookRight]
       </p>
diff --git a/ui/components/app/security-provider-banner-alert/security-provider-banner-alert.js b/ui/components/app/security-provider-banner-alert/security-provider-banner-alert.js
index b73a641554e7..87b0a42194a2 100644
--- a/ui/components/app/security-provider-banner-alert/security-provider-banner-alert.js
+++ b/ui/components/app/security-provider-banner-alert/security-provider-banner-alert.js
@@ -51,13 +51,7 @@ function SecurityProviderBannerAlert({
 
       {provider && (
         <>
-          <Text
-            marginTop={3}
-            display={Display.Flex}
-            alignItems={AlignItems.center}
-            color={Color.textAlternative}
-            variant={TextVariant.bodySm}
-          >
+          <Text marginTop={3} display={Display.Flex}>
             {t('somethingDoesntLookRight', [
               <ButtonLink
                 key={`security-provider-button-supporturl-${provider}`}
@@ -123,7 +117,7 @@ SecurityProviderBannerAlert.propTypes = {
   /** Name of the security provider */
   provider: PropTypes.oneOf(Object.values(SecurityProvider)),
 
-  /** Function to be called when the blockaid support link is clicked */
+  /** Function to be called when the support link is clicked */
   onClickSupportLink: PropTypes.func,
 };
 
diff --git a/ui/components/app/transaction-alerts/transaction-alerts.js b/ui/components/app/transaction-alerts/transaction-alerts.js
index 038563eb1762..59037f8fef7c 100644
--- a/ui/components/app/transaction-alerts/transaction-alerts.js
+++ b/ui/components/app/transaction-alerts/transaction-alerts.js
@@ -8,12 +8,7 @@ import PropTypes from 'prop-types';
 import { useSelector } from 'react-redux';
 
 import { PriorityLevels } from '../../../../shared/constants/gas';
-import {
-  submittedPendingTransactionsSelector,
-  ///: BEGIN:ONLY_INCLUDE_IN(blockaid)
-  getKnownMethodData,
-  ///: END:ONLY_INCLUDE_IN
-} from '../../../selectors';
+import { submittedPendingTransactionsSelector } from '../../../selectors';
 import { useGasFeeContext } from '../../../contexts/gasFee';
 import { useI18nContext } from '../../../hooks/useI18nContext';
 import { BannerAlert, ButtonLink, Text } from '../../component-library';
@@ -21,8 +16,6 @@ import SimulationErrorMessage from '../../ui/simulation-error-message';
 import { SEVERITIES } from '../../../helpers/constants/design-system';
 import ZENDESK_URLS from '../../../helpers/constants/zendesk-url';
 ///: BEGIN:ONLY_INCLUDE_IN(blockaid)
-import { getMethodName } from '../../../helpers/utils/metrics';
-import { TransactionType } from '../../../../shared/constants/transaction';
 import { MetaMetricsContext } from '../../../contexts/metametrics';
 ///: END:ONLY_INCLUDE_IN
 
@@ -50,10 +43,6 @@ const TransactionAlerts = ({
 
   ///: BEGIN:ONLY_INCLUDE_IN(blockaid)
   const trackEvent = useContext(MetaMetricsContext);
-  const txParams = txData?.txParams || {};
-  const methodData = useSelector((state) =>
-    getKnownMethodData(state, txParams?.data),
-  );
   ///: END:ONLY_INCLUDE_IN
 
   ///: BEGIN:ONLY_INCLUDE_IN(blockaid)
@@ -63,12 +52,6 @@ const TransactionAlerts = ({
       event: MetaMetricsEventName.ExternalLinkClicked,
       properties: {
         action: 'Confirm Screen',
-        legacy_event: true,
-        recipientKnown: null,
-        functionType:
-          'confirm' ||
-          getMethodName(methodData.name) ||
-          TransactionType.contractInteraction,
         origin: txData?.origin,
         external_link_clicked: true,
         security_alert_support_link: ZENDESK_URLS.SUPPORT_URL,