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

rpc: Explain active and internal in listdescriptors #24977

Merged
merged 1 commit into from
Apr 26, 2022

Conversation

achow101
Copy link
Member

The current help text for active and internal in listdescriptors is not particularly helpful. They require the reader to already know what those terms mean. This help text is updated to actually explain the definitions of those words in context of a descriptor wallet.

The current help text for active and internal in listdescriptors is not
particularly helpful. They require the reader to already know what those
terms mean. This help text is updated to actually explain the
definitions of those words in context of a descriptor wallet.
Copy link
Member

@jarolrod jarolrod left a comment

Choose a reason for hiding this comment

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

ACK 4637bbe

this is helpful information to add here. Sanity checked that everything looks good when calling help listdescriptors

Copy link
Contributor

@w0xlt w0xlt left a comment

Choose a reason for hiding this comment

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

ACK 4637bbe

A short definition for "ranged descriptors" could be included.

@S3RK
Copy link
Contributor

S3RK commented Apr 26, 2022

ACK 4637bbe

@fanquake fanquake merged commit f4005af into bitcoin:master Apr 26, 2022
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Apr 26, 2022
4637bbe rpc: Explain active and internal in listdescriptors (Andrew Chow)

Pull request description:

  The current help text for active and internal in listdescriptors is not particularly helpful. They require the reader to already know what those terms mean. This help text is updated to actually explain the definitions of those words in context of a descriptor wallet.

ACKs for top commit:
  S3RK:
    ACK 4637bbe
  jarolrod:
    ACK 4637bbe
  w0xlt:
    ACK bitcoin@4637bbe

Tree-SHA512: 0af2c04f3b9920799cf616ad618bde9248eb9f74cc28f443b5b0f6646deba76e9b1415aca0865ad3bcc24aa6af0e9d07ad7b7cd80f0fe80838cf847f1b944426
luke-jr pushed a commit to luke-jr/bitcoin that referenced this pull request May 21, 2022
The current help text for active and internal in listdescriptors is not
particularly helpful. They require the reader to already know what those
terms mean. This help text is updated to actually explain the
definitions of those words in context of a descriptor wallet.

Github-Pull: bitcoin#24977
Rebased-From: 4637bbe
@jb55
Copy link
Contributor

jb55 commented Oct 11, 2022 via email

@bitcoin bitcoin locked and limited conversation to collaborators Oct 11, 2023
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.

None yet

7 participants