Skip to content

Conversation

@furszy
Copy link

@furszy furszy commented Sep 30, 2021

Fixing #1235.

The QDateTime comparators are tricky as it uses the number of seconds in a day.
The day filtering window is from 00:00:00 to the last second of the day 23:59:59.

@furszy furszy added the GUI label Sep 30, 2021
@furszy furszy self-assigned this Sep 30, 2021
@furszy furszy added this to the 5.4.0 milestone Oct 1, 2021
@furszy furszy requested a review from ambassador000 October 3, 2021 15:12
Copy link
Collaborator

@Fuzzbawls Fuzzbawls left a comment

Choose a reason for hiding this comment

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

ACK 9a09dc8

Copy link

@random-zebra random-zebra left a comment

Choose a reason for hiding this comment

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

utACK 9a09dc8 and merging...

@random-zebra random-zebra merged commit 30cd9a2 into PIVX-Project:master Oct 4, 2021
furszy referenced this pull request in furszy/bitcoin-core Oct 13, 2021
@furszy furszy modified the milestones: 5.4.0, 5.3.2.1 Oct 15, 2021
furszy added a commit that referenced this pull request Oct 16, 2021
ac81ee6 GUI: bugfix, set locked label invisible for shield notes. (furszy)
6cdb6df sendmany: fix not set min depth for legacy transactions. (furszy)
74c4b9d [BUG][GUI] Refine bytes/fee/"after fee" calculation in coin-control (random-zebra)
a5d236a GUI: dashboard chart, fix last day of the month not being shown. (furszy)

Pull request description:

  List of PRs to backport to the v5.3 branch for a small and concrete bugs fixing release v5.3.2.1 (zero risk release).

  * #2568
  * #2578
  * #2592
  * #2593

ACKs for top commit:
  random-zebra:
    utACK ac81ee6
  Fuzzbawls:
    ACK ac81ee6

Tree-SHA512: 81a69e3e7ee342c4194cd34b9d230e6c2cfe5a77d3b58631aeb69406caf4e14a82f521b9df9f9953fcad6d93044e76740523b3e8eaa7d4ec6bf3079b6b2b6c95
@furszy furszy deleted the 2021_GUI_fix_chart_last_day branch November 29, 2022 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants