Skip to content

Conversation

@accraze
Copy link
Contributor

@accraze accraze commented Dec 21, 2016

This PR contains updates to the documentation for listsinceblock rpc.

fixes #8758

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is surely wrong. It should describe the meaning of the argument. And it does so in the master.

@accraze accraze force-pushed the docs-listsinceblock-rpc branch from efba947 to 3f67972 Compare December 22, 2016 17:28
@accraze
Copy link
Contributor Author

accraze commented Dec 22, 2016

@paveljanik I removed the incorrect description

@paveljanik
Copy link
Contributor

ACK 3f67972

@instagibbs
Copy link
Member

nit: maybe add "e.g., -5 implies 5 blocks ago", in case someone reads it as "-5 blocks ago"(5 blocks in the future?)

ACK 3f67972

@laanwj laanwj merged commit 3f67972 into bitcoin:master Jan 4, 2017
laanwj added a commit that referenced this pull request Jan 4, 2017
3f67972 updated listsinceblock rpc docs (accraze)
codablock pushed a commit to codablock/dash that referenced this pull request Jan 18, 2018
andvgal pushed a commit to energicryptocurrency/gen2-energi that referenced this pull request Jan 6, 2019
CryptoCentric pushed a commit to absolute-community/absolute that referenced this pull request Feb 26, 2019
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix documentations of rpc listsinceblock

6 participants