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

[0.17] backport "Validation: stop logging fake pow data" #517

Merged

Conversation

stevenroose
Copy link
Member

Based on #515.

Backport of Blockstream/liquid#8

@instagibbs
Copy link
Collaborator

utACK a657da0

@instagibbs instagibbs merged commit a657da0 into ElementsProject:elements-0.17 Mar 21, 2019
instagibbs added a commit that referenced this pull request Mar 21, 2019
a657da0 Validation: stop logging fake pow data (Sjors Provoost)

Pull request description:

  Based on #515.

  Backport of Blockstream/liquid#8

Tree-SHA512: 0f81430ca9905c4aa9b9e50ee3cd5a856132cc641dbc3346f982c2d5a023ab83cf68b64a3008f1adbda52d277d4357ed2d89e8ed6335c6288294fe5275743ba0
@stevenroose stevenroose deleted the e17-no-pow-log branch March 25, 2019 14:12
delta1 pushed a commit to delta1/elements that referenced this pull request Jun 30, 2023
…imer->setInterval() argument

f7a19ef qt,refactor: Use std::chrono in TrafficGraphWidget class (Shashwat)

Pull request description:

  The PR is a follow-up to ElementsProject#517

  - It addresses the change suggested in [this](bitcoin-core/gui#517 (review)) comment.
  - This PR changes the type of `msecsPerSample` from **int** to **std::chrono::minutes** and makes other relevant subsequent changes that were limited to the **trafficgraphwidget** file.

ACKs for top commit:
  RandyMcMillan:
    tACK f7a19ef
  hebasto:
    ACK f7a19ef
  promag:
    Code review ACK f7a19ef.

Tree-SHA512: 5094ba894f3051fc99148cb8f408fc6f9d6571188673dcb7bf24366cdfb3eaf6d4e41083685d578ad2a9fbe31cc491a5f3fa9b7c9ab6eb90e4dc1356f89ae18a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants