Skip to content
This repository has been archived by the owner on May 17, 2018. It is now read-only.

Updated to latest version #4

Open
wants to merge 91 commits into
base: master
Choose a base branch
from
Open

Updated to latest version #4

wants to merge 91 commits into from

Conversation

cgaebel
Copy link

@cgaebel cgaebel commented Oct 25, 2012

I updated network-bitcoin to match the latest version of bitcoind, as well as the latest depended-on libraries. I look forward to the next release on Hackage.

If you would like, I can continue to maintain this package from now on.

…e API calls to match the latest version of bitcoind.
@cgaebel cgaebel mentioned this pull request Oct 25, 2012
@mndrix
Copy link
Owner

mndrix commented Oct 25, 2012

Nice work. I haven't thoroughly reviewed your changes, but what I saw looked good.

That would be great if you could maintain this package from now on. I haven't been able to devote much time to maintenance lately. Do you need any form of official approval for Hackage to let you take over maintainership and upload new versions yourself? If so, let me know and I'll gladly grant the permission.

Thanks.

@cgaebel
Copy link
Author

cgaebel commented Oct 25, 2012

All I need is your blessing. Hackage doesn't have any security on who uploads what, but the community generally behaves itself. Using people's real names helps with that.

Would you mind making a commit changing the maintainer on the .cabal to "Clark Gaebel [email protected]"?

@mndrix
Copy link
Owner

mndrix commented Oct 25, 2012

Maintainer changed in .cabal in 19f9d35

Clark Gaebel and others added 30 commits April 28, 2014 08:34
Renamed SinceBlockTransaction to BlockTransaction since it is also used by listTransactions.
Added listAccounts and listTransactions.
Polished listAccounts and listTransactions.
modify API to reuse HTTP connections via Manager
Fix boolean parsing in signrawtransaction response
Rewrite `getClient` to remove `fromJust` and expose `InvalidUrlException...
changePassword called "walletpassphrase" just like unlockWallet,
when instead it should call "walletpassphrasechange" according to
https://en.bitcoin.it/wiki/Original_Bitcoin_client/API_Calls_list
When calling setGenerate while running bitcoind in regtest mode,
setGenerate is supposed to parse a [HexString] rather than Nil in some cases.
In regtest mode the setgenerate API call doesn't take a number of processors,
but rather a number of hashes to generate, and than proceeds to return these hashes.
Fixed setGenerate in regtest mode
According to [1], sendMany expects a number of minimum confirmations as its third parameter.
With null or the comment given as third parameter, as before,
bitcoind complained in regtest mode and didn't process the request as expected.
[1]: https://bitcoin.org/en/developer-reference#sendmany
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants