Merge pull request #926 from AbdulBaseerKhan/NAJ-61-deploy-contract-t…

…hrows-but-the-transaction-actually-succeeded fix: NAJ-61
near · Sep 6, 2022 · fcbd34c · fcbd34c
2 parents d097026 + a927efb
commit fcbd34c
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/packages/near-api-js/src/account.ts b/packages/near-api-js/src/account.ts
@@ -253,7 +253,8 @@ export class Account {
         }, []);
         this.printLogsAndFailures(signedTx.transaction.receiverId, flatLogs);
 
-        if (!returnError && typeof result.status === 'object' && typeof result.status.Failure === 'object') {
+        // Should be falsy if result.status.Failure is null
+        if (!returnError && typeof result.status === 'object' && typeof result.status.Failure === 'object'  && result.status.Failure !== null) {
             // if error data has error_message and error_type properties, we consider that node returned an error in the old format
             if (result.status.Failure.error_message && result.status.Failure.error_type) {
                 throw new TypedError(