gui icon indicating copy to clipboard operation
gui copied to clipboard

Network Watch tool

Open luke-jr opened this issue 2 years ago • 7 comments

Now that the crash bug (https://github.com/bitcoinknots/bitcoin/issues/4) has been tracked down, this seems ready for re-opening for Core.

(Originally https://github.com/bitcoin/bitcoin/pull/9849)

luke-jr avatar Oct 03 '21 19:10 luke-jr

Concept ACK

In general, I'm in favor of not having tons of pop-out windows. May be worth exploring this as a new tab.

If possible, can you include screenshots of this in action in the OP?

jarolrod avatar Oct 03 '21 19:10 jarolrod

Not sure how to add inline images on GitHub, but there's a sceenshot on the wiki: https://en.bitcoin.it/wiki/File:Bitcoinknots-netwatch.png

luke-jr avatar Oct 03 '21 19:10 luke-jr

As for "pop out windows", I think that's the correct UX for features like this. Having a tab in the main window just doesn't seem to fit well.

Maybe a new debug-window-alike for statistics?

luke-jr avatar Oct 03 '21 21:10 luke-jr

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Conflicts

Reviewers, this pull request conflicts with the following ones:

  • #553 (Change address / amount error background by w0xlt)
  • #537 (Point out position of invalid characters in Bech32 addresses by luke-jr)

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

DrahtBot avatar Jan 24 '22 23:01 DrahtBot

🐙 This pull request conflicts with the target branch and needs rebase.

Want to unsubscribe from rebase notifications on this pull request? Just convert this pull request to a "draft".

DrahtBot avatar Aug 30 '22 16:08 DrahtBot

concept NACK - at least, so far, I'm struggling to understand the use case for this. I'd have thought something more command line would make sense, as surely there'd be so much data that it would scroll by too quickly to be useful.

rebroad avatar Sep 03 '22 07:09 rebroad

Concept ACK if I'm understanding from the earlier PR. Suggest describing what it does and the use case (what and why) in the pull description.

jonatack avatar Sep 05 '22 12:09 jonatack

There hasn't been much activity lately and the patch still needs rebase. What is the status here?

  • Is it still relevant? ➡️ Please solve the conflicts to make it ready for review and to ensure the CI passes.
  • Is it no longer relevant? ➡️ Please close.
  • Did the author lose interest or time to work on this? ➡️ Please close it and mark it 'Up for grabs' with the label, so that it can be picked up in the future.

DrahtBot avatar Dec 04 '22 01:12 DrahtBot