From a10bb4b2fafd7093811277c50fc29456453e7623 Mon Sep 17 00:00:00 2001 From: Fabio Berger Date: Wed, 8 Nov 2017 18:59:59 -0500 Subject: Add todo comments --- src/mempool/event_watcher.ts | 2 ++ src/utils/abi_decoder.ts | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/src/mempool/event_watcher.ts b/src/mempool/event_watcher.ts index ac07badfe..cb8921cfd 100644 --- a/src/mempool/event_watcher.ts +++ b/src/mempool/event_watcher.ts @@ -47,6 +47,7 @@ export class EventWatcher { this._lastMempoolEvents = pendingEvents; } private async _getMempoolEventsAsync(): Promise { + // TODO: Allow users to listen to any number of confirmations deep, not just mempool const mempoolFilter = { fromBlock: BlockParamLiteral.Pending, toBlock: BlockParamLiteral.Pending, @@ -54,6 +55,7 @@ export class EventWatcher { const pendingEvents = await this._web3Wrapper.getLogsAsync(mempoolFilter); return pendingEvents; } + // TODO: Let's emit out own LogEntry type that has property isRemoved rather then removed private async _emitDifferencesAsync(logs: Web3.LogEntry[], isRemoved: boolean): Promise { for (const log of logs) { const logEvent = { diff --git a/src/utils/abi_decoder.ts b/src/utils/abi_decoder.ts index 247ba0e5b..840ad9be0 100644 --- a/src/utils/abi_decoder.ts +++ b/src/utils/abi_decoder.ts @@ -10,7 +10,7 @@ export class AbiDecoder { constructor(abiArrays: Web3.AbiDefinition[][]) { _.map(abiArrays, this.addABI.bind(this)); } - // This method can only decode logs from the 0x smart contracts + // This method can only decode logs from the 0x & ERC20 smart contracts public tryToDecodeLogOrNoop( log: Web3.LogEntry): LogWithDecodedArgs|RawLog { const methodId = log.topics[0]; -- cgit v1.2.3