[LIVE-1911][LIVE-1912] Feature - Add collection name resolver to eth …

…familly (#1863)

* Change nfts resolvers from Bridge type

* Update eth nft resolvers & add collection resolver

* Add loadCollectionMetadata to nft context

* Change eth NFT prepareTransaction to use collection call instead of nft metadata call

src/api/Ethereum.ts

@@ -3,7 +3,11 @@ import invariant from "invariant";
 import { BigNumber } from "bignumber.js";
 import { LedgerAPINotAvailable } from "@ledgerhq/errors";
 import JSONBigNumber from "@ledgerhq/json-bignumber";
-import type { CryptoCurrency, NFTMetadataResponse } from "../types";
+import type {
+  CryptoCurrency,
+  NFTCollectionMetadataResponse,
+  NFTMetadataResponse,
+} from "../types";
 import type { EthereumGasLimitRequest } from "../families/ethereum/types";
 import network from "../network";
 import { blockchainBaseURL } from "./Ledger";
@@ -83,6 +87,11 @@ export type NFTMetadataInput = Readonly<
     tokenId: string;
   }>
 >;
+export type NFTCollectionMetadataInput = Readonly<
+  Array<{
+    contract: string;
+  }>
+>;
 export type API = {
   getTransactions: (
     address: string,
@@ -100,6 +109,10 @@ export type API = {
     input: NFTMetadataInput,
     chainId: string
   ) => Promise<NFTMetadataResponse[]>;
+  getNFTCollectionMetadata: (
+    input: NFTCollectionMetadataInput,
+    chainId: string
+  ) => Promise<NFTCollectionMetadataResponse[]>;
   getAccountBalance: (address: string) => Promise<BigNumber>;
   roughlyEstimateGasLimit: (address: string) => Promise<BigNumber>;
   getERC20ApprovalsPerContract: (
@@ -220,6 +233,20 @@ export const apiForCurrency = (currency: CryptoCurrency): API => {
       return data;
     },
 
+    async getNFTCollectionMetadata(input, chainId) {
+      const { data }: { data: NFTCollectionMetadataResponse[] } = await network(
+        {
+          method: "POST",
+          url: `${getEnv(
+            "NFT_ETH_METADATA_SERVICE"
+          )}/v1/ethereum/${chainId}/contracts/infos`,
+          data: input,
+        }
+      );
+
+      return data;
+    },
+
     async getERC20ApprovalsPerContract(owner, contract) {
       try {
         const { data } = await network({

src/families/ethereum/bridge/js.ts

@@ -33,7 +33,7 @@ import { signOperation } from "../signOperation";
 import { modes } from "../modules";
 import postSyncPatch from "../postSyncPatch";
 import { inferDynamicRange } from "../../../range";
-import nftMetadataResolver from "../nftMetadataResolver";
+import { nftMetadata, collectionMetadata } from "../nftResolvers";
 
 const receive = makeAccountBridgeReceive();
 
@@ -211,7 +211,10 @@ const currencyBridge: CurrencyBridge = {
   preload,
   hydrate,
   scanAccounts,
-  nftMetadataResolver,
+  nftResolvers: {
+    nftMetadata,
+    collectionMetadata,
+  },
 };
 const accountBridge: AccountBridge<Transaction> = {
   createTransaction,

src/families/ethereum/modules/erc721.ts

@@ -23,11 +23,10 @@ export async function prepareTransaction(
   const { collection, collectionName, tokenIds } = transaction;
   if (collection && tokenIds && typeof collectionName === "undefined") {
     const api = apiForCurrency(account.currency);
-    const [{ status, result }] = await api.getNFTMetadata(
+    const [{ status, result }] = await api.getNFTCollectionMetadata(
       [
         {
           contract: collection,
-          tokenId: tokenIds[0],
         },
       ],
       account.currency?.ethereumLikeInfo?.chainId?.toString() || "1"

src/families/ethereum/nftResolvers.ts

@@ -0,0 +1,79 @@
+import {
+  CurrencyBridge,
+  NFTCollectionMetadataResponse,
+  NFTMetadataResponse,
+} from "../../types";
+import { getCryptoCurrencyById } from "../../currencies";
+import { metadataCallBatcher } from "../../nft";
+
+const SUPPORTED_CHAIN_IDS = new Set([
+  1, // Ethereum
+  137, // Polygon
+]);
+
+type NftResolvers = NonNullable<CurrencyBridge["nftResolvers"]>;
+
+export const nftMetadata: NftResolvers["nftMetadata"] = async ({
+  contract,
+  tokenId,
+  currencyId,
+  metadata,
+}): Promise<NFTMetadataResponse> => {
+  // This is for test/mock purposes
+  if (typeof metadata !== "undefined") {
+    return {
+      status: 200,
+      result: {
+        contract,
+        tokenId,
+        ...metadata,
+      },
+    };
+  }
+
+  const currency = getCryptoCurrencyById(currencyId);
+  const chainId = currency?.ethereumLikeInfo?.chainId;
+
+  if (!chainId || !SUPPORTED_CHAIN_IDS.has(chainId)) {
+    throw new Error("Ethereum Bridge NFT Resolver: Unsupported chainId");
+  }
+
+  const response = (await metadataCallBatcher(currency).loadNft({
+    contract,
+    tokenId,
+  })) as NFTMetadataResponse;
+
+  return response;
+};
+
+export const collectionMetadata: NftResolvers["collectionMetadata"] = async ({
+  contract,
+  currencyId,
+  metadata,
+}): Promise<NFTCollectionMetadataResponse> => {
+  // This is for test/mock purposes
+  if (typeof metadata !== "undefined") {
+    return {
+      status: 200,
+      result: {
+        contract,
+        ...metadata,
+      },
+    };
+  }
+
+  const currency = getCryptoCurrencyById(currencyId);
+  const chainId = currency?.ethereumLikeInfo?.chainId;
+
+  if (!chainId || !SUPPORTED_CHAIN_IDS.has(chainId)) {
+    throw new Error("Ethereum Bridge NFT Resolver: Unsupported chainId");
+  }
+
+  const response = (await metadataCallBatcher(currency).loadCollection({
+    contract,
+  })) as NFTCollectionMetadataResponse;
+
+  return response;
+};
+
+export default { nftMetadata, collectionMetadata };