coinbase / coinbase/mesh-bitcoin

Support for including mempool coins in /account/coins result

Open
#54 0 comments 1 reaction 0 assignees View on GitHub
enhancement
Dominant language
Go
Stars
119
Forks
127
PR merge metrics
No merged PRs in 30d

Description

**Is your feature request related to a problem? Please describe.**
While adding the [/account/coins api support](https://github.com/coinbase/rosetta-bitcoin/issues/36#issuecomment-724992022) we postponed support for including mempool coins. It would be nice to add support for this now.

**Additional context**
@patrick-ogrady I'll be happy to do a PR for this (if none is in the works currently). But I don't have much clue about how to go about doing this 😄 . If I can get some hints/tips on this I can get started right away.

Contributor guide

Open the contributing guide

Research direction

Start by reading the existing /account/coins implementation and the discussion linked from issue #36 to understand how account coins are currently gathered. Trace how mempool coins are represented and included, then use the repository's existing API tests to verify that /account/coins returns the expected result with mempool data.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
api, backend
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.