From 2a30a82183043df0ccb069af3cc72cb01a000175 Mon Sep 17 00:00:00 2001
From: Greg Nazario <greg@gnazar.io>
Date: Fri, 22 Nov 2024 22:08:59 -0500
Subject: [PATCH] Fixes coin symbols for emojicoins, and fixes showing all
 verified coins in the balances table (#935)

* [coin] Set emoji for emojicoin icon

* [coins] Ensure all verification overrides show up in balances

Verification overrides didn't show up in balances, so even if they were
validated, they wouldn't appear.  This fixes it
---
 src/components/HashButton.tsx                 | 26 ++++---
 src/pages/Account/Components/CoinsTable.tsx   | 67 +++++++++++++++++--
 .../Components/CoinBalanceChangeTable.tsx     |  6 +-
 3 files changed, 84 insertions(+), 15 deletions(-)

diff --git a/src/components/HashButton.tsx b/src/components/HashButton.tsx
index c161ea10..d5a703ce 100644
--- a/src/components/HashButton.tsx
+++ b/src/components/HashButton.tsx
@@ -221,9 +221,26 @@ function HashButtonInner({
   const open = Boolean(anchorEl);
   const id = open ? "simple-popover" : undefined;
 
+  const imgIsEmoji = img && img.match(/^\p{Emoji}+$/u);
+
   const truncateHash =
     size === "large" ? truncateAddressMiddle(hash) : truncateAddress(hash);
 
+  let icon = null;
+  if (img && imgIsEmoji) {
+    icon = (
+      <Box component="span" sx={{mr: 1, display: "flex", alignItems: "center"}}>
+        {img}
+      </Box>
+    );
+  } else if (img) {
+    icon = (
+      <Box component="span" sx={{mr: 1, display: "flex", alignItems: "center"}}>
+        <img src={img} height={20} width={20} />
+      </Box>
+    );
+  }
+
   return (
     <Box {...props}>
       <Button
@@ -248,14 +265,7 @@ function HashButtonInner({
         variant="contained"
         endIcon={<ChevronRightRoundedIcon sx={{opacity: "0.75", m: 0}} />}
       >
-        {img ? (
-          <Box
-            component="span"
-            sx={{mr: 1, display: "flex", alignItems: "center"}}
-          >
-            <img src={img} height={20} width={20} />
-          </Box>
-        ) : null}
+        {icon}
         {label ? label : truncateHash}
       </Button>
 
diff --git a/src/pages/Account/Components/CoinsTable.tsx b/src/pages/Account/Components/CoinsTable.tsx
index b48bc986..02112b87 100644
--- a/src/pages/Account/Components/CoinsTable.tsx
+++ b/src/pages/Account/Components/CoinsTable.tsx
@@ -1,11 +1,12 @@
 import * as React from "react";
+import {useEffect} from "react";
 import {
   Button,
   Stack,
   Table,
+  TableHead,
   TableRow,
   Typography,
-  TableHead,
 } from "@mui/material";
 import GeneralTableRow from "../../../components/Table/GeneralTableRow";
 import GeneralTableHeaderCell from "../../../components/Table/GeneralTableHeaderCell";
@@ -14,11 +15,14 @@ import {grey, primary} from "../../../themes/colors/aptosColorPalette";
 import GeneralTableBody from "../../../components/Table/GeneralTableBody";
 import GeneralTableCell from "../../../components/Table/GeneralTableCell";
 import {CoinDescription} from "../../../api/hooks/useGetCoinList";
-import {VerifiedCoinCell} from "../../../components/Table/VerifiedCell";
+import {
+  VerifiedCoinCell,
+  verifiedLevel,
+  VerifiedType,
+} from "../../../components/Table/VerifiedCell";
 import {getAssetSymbol} from "../../../utils";
 import {getLearnMoreTooltip} from "../../Transaction/helpers";
 import {useGlobalState} from "../../../global-config/GlobalConfig";
-import {useEffect} from "react";
 import {Network} from "@aptos-labs/ts-sdk";
 
 function CoinNameCell({name}: {name: string}) {
@@ -76,7 +80,7 @@ function CoinTypeCell({data}: {data: CoinDescriptionPlusAmount}) {
         hash={data.tokenAddress ?? data.faAddress ?? "Unknown"}
         type={getType()}
         size="large"
-        img={data.logoUrl}
+        img={data.logoUrl ? data.logoUrl : data.symbol}
       />
     </GeneralTableCell>
   );
@@ -127,12 +131,63 @@ export function CoinsTable({coins}: {coins: CoinDescriptionPlusAmount[]}) {
   }
 
   let filteredCoins = coins;
+
+  function getCoinId(coin: CoinDescriptionPlusAmount): string | null {
+    return coin.tokenAddress ?? coin.faAddress;
+  }
+
+  // TODO: This doesn't cover FAs converted from coins.  The logic for verification has gotten pretty out of hand
+  // and needs to be consolidated before going any further
+  const coinVerifications: Record<string, VerifiedType> = {};
+
+  coins.forEach((coin) => {
+    const coinId = getCoinId(coin);
+    if (coinId) {
+      coinVerifications[coinId] = verifiedLevel(
+        {
+          id: coin.tokenAddress ?? coin.faAddress ?? "Unknown",
+          known: coin.chainId !== 0,
+          isBanned: coin.isBanned,
+          isInPanoraTokenList: coin.isInPanoraTokenList,
+          symbol: coin?.panoraSymbol ?? coin.symbol,
+        },
+        state.network_name,
+      ).level;
+    }
+  });
+
   switch (verificationFilter) {
     case CoinVerificationFilterType.VERIFIED:
-      filteredCoins = coins.filter((coin) => coin.isInPanoraTokenList);
+      filteredCoins = coins.filter((coin) => {
+        const coinId = getCoinId(coin);
+        if (coinId && coinVerifications[coinId]) {
+          const level = coinVerifications[coinId];
+          return (
+            level === VerifiedType.LABS_VERIFIED ||
+            level === VerifiedType.COMMUNITY_VERIFIED ||
+            level === VerifiedType.NATIVE_TOKEN
+          );
+        } else {
+          return false;
+        }
+      });
       break;
     case CoinVerificationFilterType.RECOGNIZED:
-      filteredCoins = coins.filter((coin) => coin.chainId !== 0);
+      filteredCoins = coins.filter((coin) => {
+        const coinId = getCoinId(coin);
+        if (coinId && coinVerifications[coinId]) {
+          const level = coinVerifications[coinId];
+          return (
+            level === VerifiedType.LABS_VERIFIED ||
+            level === VerifiedType.COMMUNITY_VERIFIED ||
+            level === VerifiedType.NATIVE_TOKEN ||
+            level === VerifiedType.RECOGNIZED
+          );
+        } else {
+          return false;
+        }
+      });
+
       break;
     case CoinVerificationFilterType.ALL:
     case CoinVerificationFilterType.NONE:
diff --git a/src/pages/Transaction/Tabs/Components/CoinBalanceChangeTable.tsx b/src/pages/Transaction/Tabs/Components/CoinBalanceChangeTable.tsx
index 8d64e2ab..88b234d5 100644
--- a/src/pages/Transaction/Tabs/Components/CoinBalanceChangeTable.tsx
+++ b/src/pages/Transaction/Tabs/Components/CoinBalanceChangeTable.tsx
@@ -65,7 +65,11 @@ function TokenInfoCell({balanceChange}: BalanceChangeCellProps) {
             ? HashType.COIN
             : HashType.FUNGIBLE_ASSET
         }
-        img={balanceChange.logoUrl}
+        img={
+          balanceChange.logoUrl
+            ? balanceChange.logoUrl
+            : balanceChange.asset.symbol
+        }
       />
     </GeneralTableCell>
   );