[Explorer] Fix parsing for null values (MystenLabs#2644)

ncitron · Jun 21, 2022 · ff39738 · ff39738
1 parent 7927372
commit ff39738
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 10 deletions.
diff --git a/explorer/client/src/components/transaction-card/RecentTxCard.tsx b/explorer/client/src/components/transaction-card/RecentTxCard.tsx
@@ -220,6 +220,10 @@ function LatestTxCardAPI({ count }: { count: number }) {
                     loadState: 'fail',
                 });
                 setIsLoaded(false);
+                console.error(
+                    'Encountered error when fetching recent transactions',
+                    err
+                );
             });
 
         return () => {

diff --git a/sdk/typescript/src/rpc/client.ts b/sdk/typescript/src/rpc/client.ts
@@ -46,16 +46,20 @@ export class JsonRpcClient {
         try {
           let res: Response = await fetch(url, options);
           const text = await res.text();
-          const result = JSON.stringify(LosslessJSON.parse(text, (key : string, value : any) => {
-            if (key === "balance") return value.toString(); 
-            try {
-              if (value.isLosslessNumber) return value.valueOf();
-            } catch {
-              return value.toString();
-            }
-            return value;
-          }
-          ));
+          const result = JSON.stringify(
+            LosslessJSON.parse(text, (key: string, value: any) => {
+              if (value == null) {
+                return value;
+              }
+              if (key === 'balance') return value.toString();
+              try {
+                if (value.isLosslessNumber) return value.valueOf();
+              } catch {
+                return value.toString();
+              }
+              return value;
+            })
+          );
           if (res.ok) {
             callback(null, result);
           } else {