Merge pull request #189 from makerdao/gnosis-walletconnect-fix

explicitly add value property to tx options
makerdao · Nov 21, 2019 · ae1b2a0 · ae1b2a0
2 parents 6035dd6 + 24742dd
commit ae1b2a0
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/packages/dai/src/eth/TransactionManager.js b/packages/dai/src/eth/TransactionManager.js
@@ -47,6 +47,11 @@ export default class TransactionManager extends PublicService {
         businessObject = options.businessObject;
         delete options.businessObject;
       }
+
+      // some subproviders require a value key included with the Tx
+      if (!has(options, 'value')) {
+        options.value = 0;
+      }
     } else {
       options = {};
     }