Fix: Allow use of custodian without any browser set (#512)

Signed-off-by: jaime-iobermudez <[email protected]>

Author: jaime-iobermudez

sdk/src/app/service/transaction/TransactionService.ts

@@ -257,21 +257,12 @@ export default class TransactionService extends Service {
         LogService.logTrace('HWALLETCONNECT TransactionAdapter');
         return Injectable.resolve(HederaWalletConnectTransactionAdapter);
       case SupportedWallets.DFNS:
-        if (!Injectable.isWeb()) {
-          throw new InvalidWalletTypeError();
-        }
         LogService.logTrace('DFNS TransactionAdapter');
         return Injectable.resolve(DFNSTransactionAdapter);
       case SupportedWallets.FIREBLOCKS:
-        if (!Injectable.isWeb()) {
-          throw new InvalidWalletTypeError();
-        }
         LogService.logTrace('FIREBLOCKS TransactionAdapter');
         return Injectable.resolve(FireblocksTransactionAdapter);
       case SupportedWallets.AWSKMS:
-        if (!Injectable.isWeb()) {
-          throw new InvalidWalletTypeError();
-        }
         LogService.logTrace('AWSKMS TransactionAdapter');
         return Injectable.resolve(AWSKMSTransactionAdapter);
       default:

sdk/src/core/Injectable.ts

@@ -1202,9 +1202,13 @@ export default class Injectable {
 
   static registerTransactionAdapterInstances(): TransactionAdapter[] {
     const adapters: TransactionAdapter[] = [];
+    if (this.isWeb()) {
+      adapters.push(
+        Injectable.resolve(RPCTransactionAdapter),
+        Injectable.resolve(HederaWalletConnectTransactionAdapter),
+      );
+    }
     adapters.push(
-      Injectable.resolve(RPCTransactionAdapter),
-      Injectable.resolve(HederaWalletConnectTransactionAdapter),
       Injectable.resolve(DFNSTransactionAdapter),
       Injectable.resolve(FireblocksTransactionAdapter),
       Injectable.resolve(AWSKMSTransactionAdapter),