From 85af1ddd50fbac7ccffb5eb4fb0a5df4b3e984e2 Mon Sep 17 00:00:00 2001
From: unclebill <billbill290@gmail.com>
Date: Sat, 9 Jul 2022 12:22:50 +0800
Subject: [PATCH 1/2] fix: hide buy button on NFT trending

---
 .../plugins/Trader/SNSAdaptor/trending/TrendingViewDeck.tsx   | 4 ++--
 packages/plugins/GoPlusSecurity/src/SNSAdaptor/index.tsx      | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/packages/mask/src/plugins/Trader/SNSAdaptor/trending/TrendingViewDeck.tsx b/packages/mask/src/plugins/Trader/SNSAdaptor/trending/TrendingViewDeck.tsx
index 45179cb1a0bf..549f9909e4ae 100644
--- a/packages/mask/src/plugins/Trader/SNSAdaptor/trending/TrendingViewDeck.tsx
+++ b/packages/mask/src/plugins/Trader/SNSAdaptor/trending/TrendingViewDeck.tsx
@@ -157,10 +157,10 @@ export function TrendingViewDeck(props: TrendingViewDeckProps) {
         isTokenSecurityEnable,
     )
 
+    const isBuyable = !isNFT && transakPluginEnabled && !transakIsMinimalMode && trending.coin.symbol && isAllowanceCoin
     const onBuyButtonClicked = useCallback(() => {
         setBuyDialog({
             open: true,
-            // @ts-ignore
             code: coin.symbol,
             address: account,
         })
@@ -293,7 +293,7 @@ export function TrendingViewDeck(props: TrendingViewDeckProps) {
                         </Stack>
                     </Stack>
                     <Stack>
-                        {transakPluginEnabled && !transakIsMinimalMode && trending.coin.symbol && isAllowanceCoin ? (
+                        {isBuyable ? (
                             <Button
                                 style={{
                                     background: theme.palette.maskColor.dark,
diff --git a/packages/plugins/GoPlusSecurity/src/SNSAdaptor/index.tsx b/packages/plugins/GoPlusSecurity/src/SNSAdaptor/index.tsx
index 0194a6b1b098..f68bdcc78cf6 100644
--- a/packages/plugins/GoPlusSecurity/src/SNSAdaptor/index.tsx
+++ b/packages/plugins/GoPlusSecurity/src/SNSAdaptor/index.tsx
@@ -33,7 +33,7 @@ const sns: Plugin.SNSAdaptor.Definition = {
                 },
                 name,
                 icon,
-                iconFilterColor,
+                iconFilterColor: '',
                 appBoardSortingDefaultPriority: 14,
                 category: 'dapp',
                 marketListSortingPriority: 16,

From bbe142e41265574a4704cdb04af2ca1f816cb362 Mon Sep 17 00:00:00 2001
From: unclebill <billbill290@gmail.com>
Date: Sat, 9 Jul 2022 12:32:12 +0800
Subject: [PATCH 2/2] fixup! fix: hide buy button on NFT trending

---
 packages/mask/src/plugins/Trader/trending/useTrending.ts | 2 +-
 packages/web3-providers/src/coingecko/base-api.ts        | 2 +-
 packages/web3-providers/src/coingecko/index.ts           | 2 ++
 3 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/packages/mask/src/plugins/Trader/trending/useTrending.ts b/packages/mask/src/plugins/Trader/trending/useTrending.ts
index 8cd6020b561d..346482b103cd 100644
--- a/packages/mask/src/plugins/Trader/trending/useTrending.ts
+++ b/packages/mask/src/plugins/Trader/trending/useTrending.ts
@@ -50,7 +50,7 @@ export function useTrendingById(id: string, dataProvider: DataProvider) {
     } = useAsync(async () => {
         if (!id) return null
         if (!currency) return null
-        return PluginTraderRPC.getCoinTrendingById(chainId, id, currency, dataProvider)
+        return PluginTraderRPC.getCoinTrendingById(chainId, id, currency, dataProvider).catch(() => null)
     }, [chainId, dataProvider, currency?.id, id])
 
     const { value: detailedToken } = useFungibleToken(NetworkPluginID.PLUGIN_EVM, trending?.coin.contract_address)
diff --git a/packages/web3-providers/src/coingecko/base-api.ts b/packages/web3-providers/src/coingecko/base-api.ts
index bcf3cbf54fd5..3ecaecc79d8a 100644
--- a/packages/web3-providers/src/coingecko/base-api.ts
+++ b/packages/web3-providers/src/coingecko/base-api.ts
@@ -103,7 +103,7 @@ export interface CoinInfo {
 }
 
 export async function getCoinInfo(coinId: string) {
-    return fetchJSON<CoinInfo>(
+    return fetchJSON<CoinInfo | { error: string }>(
         `${COINGECKO_URL_BASE}/coins/${coinId}?developer_data=false&community_data=false&tickers=true`,
         { cache: getTraderAllAPICachedFlag() },
     )
diff --git a/packages/web3-providers/src/coingecko/index.ts b/packages/web3-providers/src/coingecko/index.ts
index c88a29637929..50b10c0c47f6 100644
--- a/packages/web3-providers/src/coingecko/index.ts
+++ b/packages/web3-providers/src/coingecko/index.ts
@@ -53,6 +53,8 @@ export class CoinGeckoAPI implements PriceAPI.Provider, TrendingAPI.Provider<Cha
 
     async getCoinTrending(chainId: ChainId, id: string, currency: TrendingAPI.Currency): Promise<TrendingAPI.Trending> {
         const info = await getCoinInfo(id)
+        if ('error' in info) throw new Error(info.error)
+
         const platform_url = `https://www.coingecko.com/en/coins/${info.id}`
         const twitter_url = info.links.twitter_screen_name
             ? `https://twitter.com/${info.links.twitter_screen_name}`