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

doc: Clarify listpeerchannels documentation. #7155

Conversation

evansmj
Copy link
Contributor

@evansmj evansmj commented Mar 17, 2024

This is to clarify the listpeerchannels documentation. I had been reading it as though listpeerchannels returns all channels in the network, which while developing a frontend did not make sense so I spent time testing in startup_regtest.sh to figure out what it really does. I think this clarification would help others in the future.

@evansmj evansmj changed the title Clarify listpeerchannels documentation. doc: Clarify listpeerchannels documentation. Mar 17, 2024
@evansmj evansmj marked this pull request as draft March 19, 2024 00:29
@evansmj
Copy link
Contributor Author

evansmj commented Mar 19, 2024

Wait until #6995 is merged

@@ -9,7 +9,7 @@ SYNOPSIS
DESCRIPTION
-----------

The **listpeerchannels** RPC command returns data on channels of the network, with the possibility to filter the channels by node id.
The **listpeerchannels** RPC command returns data on this node's channels, with the possibility to filter the channels by the id of a remote node.
Copy link
Contributor

Choose a reason for hiding this comment

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

idk if this is clear, maybe it is. However, listpeerchannels return the list of channels inside the network divided by peers.

e.g:
peer -> channels
peer -> channels

@ShahanaFarooqui
Copy link
Collaborator

Closing in favour of PR #7326.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants