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.

Sign up for GitHub

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

Jump to bottom

btcctl: Add getwalletinfo command #471

Merged
merged 1 commit into from Jul 24, 2015
Merged

btcctl: Add getwalletinfo command #471

merged 1 commit into from Jul 24, 2015

Conversation

drazisil
Copy link
Contributor

Adds wallet server command getwalletinfo

@davecgh davecgh changed the title added getwalletinfo cmd btcctl: Add getwalletinfo command Jul 24, 2015
@davecgh
Copy link
Member

davecgh commented Jul 24, 2015

Thanks for the PR. Looks good.

OK

@dajohi
Copy link
Member

dajohi commented Jul 24, 2015

OK.. but can you add the Result too?

See https://github.com/bitcoin/bitcoin/blob/master/src/wallet/rpcwallet.cpp#L2184-L2224

@dajohi
Copy link
Member

dajohi commented Jul 24, 2015

but OK on this diff :)

@davecgh
Copy link
Member

davecgh commented Jul 24, 2015

The result isn't needed for btcctl. Once btcwallet implements the command, the result can be added then.

@conformal-deploy conformal-deploy merged commit b160644 into btcsuite:master Jul 24, 2015
@drazisil drazisil deleted the feature_cmd_getwalletinfo branch July 24, 2015 19:39
davecgh pushed a commit to davecgh/btcd that referenced this pull request Nov 17, 2016
@dajohi @alexlyp
blockchain: LogBlockHeight only needs a wire.MsgBlock.. (btcsuite#471)
563f4d1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@drazisil @davecgh @dajohi @conformal-deploy