Merge pull request #13 from bitcoinerlab/mempool-blockheight-0

fix: use 0 for mempool blockheight accross all implementations

Author: landabaso

package-lock.json

@@ -2,7 +2,7 @@
   "name": "@bitcoinerlab/explorer",
   "description": "Bitcoin Blockchain Explorer: Client Interface featuring Esplora and Electrum Implementations.",
   "homepage": "https://github.com/bitcoinerlab/explorer",
-  "version": "0.3.2",
+  "version": "0.3.3",
   "author": "Jose-Luis Landabaso",
   "license": "MIT",
   "prettier": "@bitcoinerlab/configs/prettierConfig.json",

package.json

@@ -416,7 +416,10 @@ export class ElectrumExplorer implements Explorer {
 
     const transactionHistory = history.map(({ tx_hash, height }) => {
       const txId = tx_hash;
-      const blockHeight: number = height || 0;
+      //Electrum returns -1 for mempool, however we use blockHeight = 0 to
+      //denote mempool
+      const blockHeight: number =
+        parseInt(height) === -1 ? 0 : parseInt(height);
       if (blockHeight > this.#tipBlockHeight) {
         console.warn(
           `tx ${tx_hash} block height ${blockHeight} larger than the tip ${this.#tipBlockHeight}`