fix(Trade): expose txHash property, only trigger watch after trade ha…

…s been updated (#264)
blockchain · Sep 2, 2016 · 31eccee · 31eccee
1 parent 81e9fc6
commit 31eccee
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/src/coinify/trade.js b/src/coinify/trade.js
@@ -120,6 +120,10 @@ Object.defineProperties(CoinifyTrade.prototype, {
         return this.outCurrency === 'BTC';
       }
     }
+  },
+  'txHash': {
+    configurable: false,
+    get: function () { return this._txHash || null; }
   }
 });
 
@@ -333,11 +337,13 @@ CoinifyTrade.prototype._monitorAddress = function () {
   };
 
   var tradeWasPaid = function (amount) {
-    self._watchAddressResolve && self._watchAddressResolve(amount);
-
+    var resolve = function () {
+      self._watchAddressResolve && self._watchAddressResolve(amount);
+    };
     self.refresh()
       .then(CoinifyTrade._getTransactionHash)
-      .then(saveTrade);
+      .then(saveTrade)
+      .then(resolve);
   };
 
   self._coinify.delegate.monitorAddress(self.receiveAddress, function (amount) {