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

[GUI] [BUG] set locked label invisible for shield notes. #2593

Merged
merged 1 commit into from
Oct 15, 2021

Conversation

furszy
Copy link

@furszy furszy commented Oct 12, 2021

We don't support shield locked notes functionality yet. As we are reusing the same coin control dialog, once we open the dialog for shield notes, the locked coins label can still be, invalidly, showing the previous label value for regular utxo.

We don't have support for the functionality yet.
@furszy furszy self-assigned this Oct 12, 2021
@furszy
Copy link
Author

furszy commented Oct 12, 2021

Just for an informational purpose. This problem was reported by ambassador ☕. Thanks for continue testing the wallet and providing feedback 🤘.

@furszy furszy added this to the 5.4.0 milestone Oct 12, 2021
@ambassador000
Copy link

As we spoke already about it, due to current testing environment limits, I'll be able to confirm if this PR solves the issue entirely post v5.3.2.1 release.

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.

ACK ff846a9

@furszy furszy requested a review from Fuzzbawls October 14, 2021 13:28
@furszy furszy modified the milestones: 5.4.0, 5.3.2.1 Oct 15, 2021
@random-zebra random-zebra merged commit 5d9db44 into PIVX-Project:master 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_coincontrol 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