From 9ba6443637d34288dd267592e156f154ce609e91 Mon Sep 17 00:00:00 2001
From: Brendon Votteler <bvotteler@users.noreply.github.com>
Date: Fri, 19 Aug 2022 16:31:06 +0200
Subject: [PATCH] fix: do not throw on dashboard for non-token collateral vault
 - filter & log instead

---
 .../sub-pages/Vaults/VaultsTable/index.tsx    | 115 +++++++++---------
 1 file changed, 60 insertions(+), 55 deletions(-)

diff --git a/src/pages/Dashboard/sub-pages/Vaults/VaultsTable/index.tsx b/src/pages/Dashboard/sub-pages/Vaults/VaultsTable/index.tsx
index 940e3e6896..e77d07dd8b 100644
--- a/src/pages/Dashboard/sub-pages/Vaults/VaultsTable/index.tsx
+++ b/src/pages/Dashboard/sub-pages/Vaults/VaultsTable/index.tsx
@@ -271,64 +271,69 @@ const VaultsTable = (): JSX.Element => {
       collateralThresholds &&
       currentActiveBlockNumber
     ) {
-      const rawVaults = filteredVaults.map((vaultExt) => {
-        const collateral = vaultExt.id.currencies.collateral;
-        if (collateral.isToken === false) {
-          throw new Error('Non token collateral is not supported!');
-        }
-        const collateralTokenSymbol = collateral.asToken.type as CollateralIdLiteral;
-
-        const collateralLiquidationThreshold = collateralThresholds[collateralTokenSymbol].liquidationThreshold;
-        const collateralSecureThreshold = collateralThresholds[collateralTokenSymbol].secureThreshold;
-        let btcToCollateralTokenRate: BTCToCollateralTokenRate;
-        switch (collateralTokenSymbol) {
-          case RELAY_CHAIN_NATIVE_TOKEN_SYMBOL: {
-            btcToCollateralTokenRate = btcToRelayChainNativeTokenRate;
-            break;
+      const rawVaults = filteredVaults
+        .filter((vaultExt) => {
+          if (vaultExt.id.currencies.collateral.isToken) {
+            return true;
           }
-          case GOVERNANCE_TOKEN_SYMBOL: {
-            btcToCollateralTokenRate = btcToGovernanceTokenRate;
-            break;
+          console.warn("Unsupported non token collateral vault found and removed from vaults list");
+          return false;
+        })
+        .map((vaultExt) => {
+          const collateral = vaultExt.id.currencies.collateral;
+          const collateralTokenSymbol = collateral.asToken.type as CollateralIdLiteral;
+
+          const collateralLiquidationThreshold = collateralThresholds[collateralTokenSymbol].liquidationThreshold;
+          const collateralSecureThreshold = collateralThresholds[collateralTokenSymbol].secureThreshold;
+          let btcToCollateralTokenRate: BTCToCollateralTokenRate;
+          switch (collateralTokenSymbol) {
+            case RELAY_CHAIN_NATIVE_TOKEN_SYMBOL: {
+              btcToCollateralTokenRate = btcToRelayChainNativeTokenRate;
+              break;
+            }
+            case GOVERNANCE_TOKEN_SYMBOL: {
+              btcToCollateralTokenRate = btcToGovernanceTokenRate;
+              break;
+            }
+            default:
+              throw new Error('Something went wrong with collateralTokenType!');
           }
-          default:
-            throw new Error('Something went wrong with collateralTokenType!');
-        }
-
-        const statusLabel = getVaultStatusLabel(
-          vaultExt,
-          currentActiveBlockNumber,
-          collateralLiquidationThreshold,
-          collateralSecureThreshold,
-          btcToCollateralTokenRate,
-          t
-        );
-
-        const vaultCollateral = vaultExt.backingCollateral;
-        const settledTokens = vaultExt.issuedTokens;
-        const settledCollateralization = getCollateralization(vaultCollateral, settledTokens, btcToCollateralTokenRate);
-        const unsettledTokens = vaultExt.toBeIssuedTokens;
-        const unsettledCollateralization = getCollateralization(
-          vaultCollateral,
-          unsettledTokens.add(settledTokens),
-          btcToCollateralTokenRate
-        );
 
-        return {
-          [Accessor.VaultId]: vaultExt.id.accountId.toString(),
-          [Accessor.Collateral]: collateralTokenSymbol,
-          // TODO: fetch collateral reserved
-          [Accessor.LockedCollateralTokenAmount]: `${displayMonetaryAmount(vaultCollateral)} ${collateralTokenSymbol}`,
-          [Accessor.LockedBTCAmount]: settledTokens,
-          [Accessor.PendingBTCAmount]: displayMonetaryAmount(unsettledTokens),
-          [Accessor.CollateralizationUI]: (
-            <CollateralizationCell
-              settledCollateralization={settledCollateralization}
-              unsettledCollateralization={unsettledCollateralization}
-              collateralSecureThreshold={collateralSecureThreshold}
-            />
-          ),
-          [Accessor.Status]: statusLabel
-        };
+          const statusLabel = getVaultStatusLabel(
+            vaultExt,
+            currentActiveBlockNumber,
+            collateralLiquidationThreshold,
+            collateralSecureThreshold,
+            btcToCollateralTokenRate,
+            t
+          );
+
+          const vaultCollateral = vaultExt.backingCollateral;
+          const settledTokens = vaultExt.issuedTokens;
+          const settledCollateralization = getCollateralization(vaultCollateral, settledTokens, btcToCollateralTokenRate);
+          const unsettledTokens = vaultExt.toBeIssuedTokens;
+          const unsettledCollateralization = getCollateralization(
+            vaultCollateral,
+            unsettledTokens.add(settledTokens),
+            btcToCollateralTokenRate
+          );
+
+          return {
+            [Accessor.VaultId]: vaultExt.id.accountId.toString(),
+            [Accessor.Collateral]: collateralTokenSymbol,
+            // TODO: fetch collateral reserved
+            [Accessor.LockedCollateralTokenAmount]: `${displayMonetaryAmount(vaultCollateral)} ${collateralTokenSymbol}`,
+            [Accessor.LockedBTCAmount]: settledTokens,
+            [Accessor.PendingBTCAmount]: displayMonetaryAmount(unsettledTokens),
+            [Accessor.CollateralizationUI]: (
+              <CollateralizationCell
+                settledCollateralization={settledCollateralization}
+                unsettledCollateralization={unsettledCollateralization}
+                collateralSecureThreshold={collateralSecureThreshold}
+              />
+            ),
+            [Accessor.Status]: statusLabel
+          };
       });
 
       const sortedVaults = rawVaults.sort((vaultA, vaultB) => {