Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cleanup and Merge of Bitcore-integration #184

Open
wants to merge 8 commits into
base: improvement/explorer
Choose a base branch
from

Conversation

nimbosa
Copy link
Member

@nimbosa nimbosa commented Sep 7, 2018

Weighing in on the rationale to keep this as a separate branch and an analogous release instead of merging back to master.

These two:

seem to be more future-proof approaches,
per the discussion here: bitcoin/bitcoin#10370
as noted by ch4ot1c here: #169

bitcartel and others added 8 commits February 28, 2018 14:12
If generated is true, the unspent transaction output is from a
coinbase transaction and can only be sent to a shielded address.
bitcore integration: cleanup and fix of mixed use of tabs and spaces in favor of spaces consistently
@ch4ot1c
Copy link
Collaborator

ch4ot1c commented Sep 17, 2018

(keeping it as a separate branch is lighter for most clients, and we are seemingly planning to deprecate it. And it requires txindex=1)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants