Rename fetchTransactions to fetchAddressHistory

2025-04-09 02:03:27 +00:00
parent a662d5cc7e
commit 05f858549b
6 changed files with 27 additions and 15 deletions
@@ -83,7 +83,15 @@ export abstract class BaseBlockchain extends EventEmitter<BlockchainEvents> {
   * @returns {Promise<BlockchainTransactions>} A promise that resolves with the list of transactions
   * @throws {Error} If the address is invalid or the query fails
   */
-  abstract fetchTransactions(address: string): Promise<BlockchainTransactions>;
+  abstract fetchAddressHistory(
+    address: string,
+  ): Promise<BlockchainTransactions>;
+
+  // TODO: We may have a fetchTokenHistory in future, but Fulcrum does not currently support indexing tokens.
+  //       There are actually a few functional discrepancies with the adapters: different adapters grant different capabilities.
+  //       For example, Chaingraph can fetch token and address history. But BCHN P2P network cannot even fetch address history.
+  //       We might want to handle this by having multiple BlockchainInterfaces (e.g. IBlockchainAddressSubscriptions) that describe capabilities.
+  //       But we should return to that later.

  /**
   * Broadcasts a raw transaction to the blockchain network.