Skip to content

Conversation

johnny9
Copy link
Collaborator

@johnny9 johnny9 commented Jul 31, 2025

The upstream PeerTableModel does not expose its startAutoRefresh and stopAutoRefresh methods as invokable so the PeerTableQmlModel is introduced to make these methods invokable in qml.

fixes #482

The upstream PeerTableModel does not expose its startAutoRefresh
and stopAutoRefresh methods as invokable so the PeerTableQmlModel
is introduced to make these methods invokable in qml.
Q_INVOKABLE void stopAutoRefresh();
};

#endif // PEERTABLEQMLMODEL_H
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
#endif // PEERTABLEQMLMODEL_H
#endif // BITCOIN_QML_MODELS_PEERTABLEQMLMODEL_H

@johnny9 johnny9 force-pushed the peers-qml branch 2 times, most recently from 14c2540 to a0d6179 Compare August 1, 2025 16:55
@johnny9
Copy link
Collaborator Author

johnny9 commented Aug 1, 2025

Update from 14c2540 to a0d6179

  • Fixed include guard comment in peertableqmlmodel.h
  • Added copyright comment to the top of the new files

Copy link
Member

@hebasto hebasto left a comment

Choose a reason for hiding this comment

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

ACK a0d6179, tested on Ubuntu 24.04.

@hebasto hebasto merged commit 04b2a1f into bitcoin-core:qt6 Aug 2, 2025
2 checks passed
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.

qt6: Settings --> Peers not working
2 participants