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] Better start single MN error description. #1732

Merged
merged 1 commit into from
Jul 10, 2020

Conversation

furszy
Copy link

@furszy furszy commented Jul 6, 2020

Built on top of #1719 .

Fixing/Improving a bad error notification string showed in the GUI due the CMasternodeBroadcast::Create calling CWallet::GetMasternodeVinAndKeys and returning to the GUI the same string error output that is being printed to the log (with all of the extra data, including function names, service etc..).

Can be easily tested creating a masternode, then not waiting for the 15 confirmations and trying to start the node via the menu single masternode start (click on the masternode row and then press start).
Without the last commit, will popup a bad error notification text.

@furszy furszy self-assigned this Jul 6, 2020
random-zebra
random-zebra previously approved these changes Jul 6, 2020
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 3908123

@random-zebra random-zebra added this to the 4.2.0 milestone Jul 6, 2020
@Fuzzbawls Fuzzbawls changed the title GUI: Better start single MN error description. [GUI] Better start single MN error description. Jul 8, 2020
@Fuzzbawls
Copy link
Collaborator

can be rebased now that #1719 has been merged

@furszy
Copy link
Author

furszy commented Jul 8, 2020

rebased

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 1d42014

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.

utACK 1d42014

@Fuzzbawls Fuzzbawls merged commit be99df3 into PIVX-Project:master Jul 10, 2020
@furszy furszy deleted the 2020_gui_mn_start_error branch November 29, 2022 14:21
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.

None yet

3 participants