Skip to content

Introduce PeersIndicator component, Add to BlockClock#236

Merged
hebasto merged 4 commits into
bitcoin-core:mainfrom
jarolrod:peersindicator-component
Feb 9, 2023
Merged

Introduce PeersIndicator component, Add to BlockClock#236
hebasto merged 4 commits into
bitcoin-core:mainfrom
jarolrod:peersindicator-component

Conversation

@jarolrod
Copy link
Copy Markdown
Contributor

@jarolrod jarolrod commented Jan 31, 2023

This is a rebased version of, and replaces, #127

Introduces the actual PeersIndicator component and adds it to the BlockClock

Dark Mode Light Mode
Screen Shot 2023-01-31 at 12 48 23 AM Screen Shot 2023-01-31 at 12 49 09 AM

Windows
Intel macOS
Apple Silicon macOS
ARM64 Android

This was referenced Feb 1, 2023
Copy link
Copy Markdown
Collaborator

@johnny9 johnny9 left a comment

Choose a reason for hiding this comment

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

ACK c7c73b8

I'm assuming changing the node interface is ok to do. The changes themselves look good and I ran and tested on mainnet and see the indicator updating as expected with no errors.

@hebasto hebasto merged commit 22838d2 into bitcoin-core:main Feb 9, 2023
johnny9 pushed a commit to johnny9/bitcoin-core-app that referenced this pull request Jul 4, 2025
…dd to BlockClock

fc34996 qml: add PeersIndicator to BlockClock (jarolrod)
5d0ea0e qml: Add `PeersIndicator` component (Hennadii Stepanov)
f5c2cd0 qml: Add `NodeModel::numOutboundPeers` property (Hennadii Stepanov)
475c63ef4752f7b79eca647072208f3a5be8d56e net: Notify about connection number by type (Hennadii Stepanov)

Pull request description:

  This is a rebased version of, and replaces, bitcoin-core/gui-qml#127

  Introduces the actual `PeersIndicator` component and adds it to the `BlockClock`

  | Dark Mode | Light Mode |
  | --------- | ---------- |
  | <img width="752" alt="Screen Shot 2023-01-31 at 12 48 23 AM" src="https://user-images.githubusercontent.com/23396902/215676831-6573a94f-2030-4bf2-b45b-9d325be42600.png"> | <img width="752" alt="Screen Shot 2023-01-31 at 12 49 09 AM" src="https://user-images.githubusercontent.com/23396902/215676882-96f82c03-43ff-449d-b5d9-d7016fb0540b.png"> |

  [![Windows](https://img.shields.io/badge/OS-Windows-green)](https://api.cirrus-ci.com/v1/artifact/github/bitcoin-core/gui-qml/win64/insecure_win_gui.zip?branch=pull/236)
  [![Intel macOS](https://img.shields.io/badge/OS-Intel%20macOS-green)](https://api.cirrus-ci.com/v1/artifact/github/bitcoin-core/gui-qml/macos/insecure_mac_gui.zip?branch=pull/236)
  [![Apple Silicon macOS](https://img.shields.io/badge/OS-Apple%20Silicon%20macOS-green)](https://api.cirrus-ci.com/v1/artifact/github/bitcoin-core/gui-qml/macos_arm64/insecure_mac_arm64_gui.zip?branch=pull/236)
  [![ARM64 Android](https://img.shields.io/badge/OS-Android-green)](https://api.cirrus-ci.com/v1/artifact/github/bitcoin-core/gui-qml/android/insecure_android_apk.zip?branch=pull/236)

ACKs for top commit:
  johnny9:
    ACK fc34996

Tree-SHA512: f5af2cd53bcecdb9441f412e94ad6557c48aad7998c2da69f1e0174682a0761151ca72b27a094b12d3f9055dff61694b9205297f5a4db9559bd0ed0ff3c4fdca
tx-signer450 added a commit to tx-signer450/gui-qml that referenced this pull request Oct 20, 2025
…dd to BlockClock

fc34996357b8cb7830b1fbc26bc8f91575f88cff qml: add PeersIndicator to BlockClock (jarolrod)
5d0ea0ed5768109a60a86ec1a22e146812a1e021 qml: Add `PeersIndicator` component (Hennadii Stepanov)
f5c2cd044833e8365224f382692d27ddcfa8f3d9 qml: Add `NodeModel::numOutboundPeers` property (Hennadii Stepanov)
475c63ef4752f7b79eca647072208f3a5be8d56e net: Notify about connection number by type (Hennadii Stepanov)

Pull request description:

  This is a rebased version of, and replaces, bitcoin-core/gui-qml#127

  Introduces the actual `PeersIndicator` component and adds it to the `BlockClock`

  | Dark Mode | Light Mode |
  | --------- | ---------- |
  | <img width="752" alt="Screen Shot 2023-01-31 at 12 48 23 AM" src="https://user-images.githubusercontent.com/23396902/215676831-6573a94f-2030-4bf2-b45b-9d325be42600.png"> | <img width="752" alt="Screen Shot 2023-01-31 at 12 49 09 AM" src="https://user-images.githubusercontent.com/23396902/215676882-96f82c03-43ff-449d-b5d9-d7016fb0540b.png"> |

  [![Windows](https://img.shields.io/badge/OS-Windows-green)](https://api.cirrus-ci.com/v1/artifact/github/bitcoin-core/gui-qml/win64/insecure_win_gui.zip?branch=pull/236)
  [![Intel macOS](https://img.shields.io/badge/OS-Intel%20macOS-green)](https://api.cirrus-ci.com/v1/artifact/github/bitcoin-core/gui-qml/macos/insecure_mac_gui.zip?branch=pull/236)
  [![Apple Silicon macOS](https://img.shields.io/badge/OS-Apple%20Silicon%20macOS-green)](https://api.cirrus-ci.com/v1/artifact/github/bitcoin-core/gui-qml/macos_arm64/insecure_mac_arm64_gui.zip?branch=pull/236)
  [![ARM64 Android](https://img.shields.io/badge/OS-Android-green)](https://api.cirrus-ci.com/v1/artifact/github/bitcoin-core/gui-qml/android/insecure_android_apk.zip?branch=pull/236)

ACKs for top commit:
  johnny9:
    ACK fc34996357b8cb7830b1fbc26bc8f91575f88cff

Tree-SHA512: f5af2cd53bcecdb9441f412e94ad6557c48aad7998c2da69f1e0174682a0761151ca72b27a094b12d3f9055dff61694b9205297f5a4db9559bd0ed0ff3c4fdca
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