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

Already on GitHub? Sign in to your account

Document direct headers annoucement in block broadcasting guide. #1740

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants
Contributor

jimpo commented Aug 10, 2017

BIP 130 added a new standard protocol for announcing new blocks that is not yet documented in the Bitcoin Developer Guide.

@wbnns wbnns self-assigned this Aug 10, 2017

@wbnns wbnns added the Under Review label Aug 10, 2017

@wbnns wbnns deleted a comment from cjennyc28 Aug 10, 2017

@wbnns wbnns deleted a comment from cjennyc28 Aug 10, 2017

@wbnns wbnns deleted a comment from cjennyc28 Aug 10, 2017

@wbnns wbnns deleted a comment from cjennyc28 Aug 10, 2017

@wbnns wbnns deleted a comment from cjennyc28 Aug 10, 2017

@wbnns wbnns deleted a comment from cjennyc28 Aug 10, 2017

@wbnns wbnns deleted a comment from cjennyc28 Aug 10, 2017

Gemfile.lock
@@ -37,7 +37,7 @@ GEM
execjs
multi_json (>= 1.3)
rake
- json (1.8.3)
+ json (2.1.0)
@kuzzmi

kuzzmi Aug 10, 2017

Contributor

Dependencies upgrade is already a part of another PR: #1715.
Could you please revert this change, so Travis CI doesn't break? Thanks!

Document direct headers annoucement in block broadcasting guide.
BIP 130 added a new standard protocol for announcing new blocks that
is not yet documented in the Bitcoin Developer Guide.

@wbnns wbnns added On Hold and removed Under Review labels Sep 10, 2017

Contributor

wbnns commented Sep 16, 2017

@jimpo Heya! I went ahead and cherry picked this into #1805 so you won't have to mess around with rebasing. Congrats on your first contribution to the repo and thanks a bunch!

@wbnns wbnns closed this Sep 16, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment