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

fix: list-connections #4841

Merged
merged 1 commit into from Oct 21, 2022
Merged

Conversation

Cifko
Copy link
Contributor

@Cifko Cifko commented Oct 21, 2022

Description

List-connections command prints wallets only if there are base node connections.

How Has This Been Tested?

Manually.

Copy link
Member

@sdbondi sdbondi left a comment

Choose a reason for hiding this comment

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

🤣 good catch

@CjS77 CjS77 added the P-acks_required Process - Requires more ACKs or utACKs label Oct 21, 2022
@sdbondi
Copy link
Member

sdbondi commented Oct 21, 2022

ACK

@CjS77 CjS77 removed the P-acks_required Process - Requires more ACKs or utACKs label Oct 21, 2022
@sdbondi sdbondi added the P-merge Process - Queued for merging label Oct 21, 2022
@sdbondi sdbondi merged commit 23b2c9a into tari-project:development Oct 21, 2022
@Cifko Cifko deleted the fix-list-connections branch April 18, 2023 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P-merge Process - Queued for merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants