Skip to content

Conversation

@furszy
Copy link

@furszy furszy commented Jan 5, 2021

Back porting PRs from master:

#2100
#2111
#2112
#2113
#2114
#2115
#2116
#2119
#2120
#2121
#2122
#2123
#2125

furszy and others added 26 commits January 5, 2021 18:22
serialized size of std::vector (inputs, outputs, shieldspends,
shieldoutputs) takes more than one byte if the vector has more than 253
elements.
Github-Pull: bitcoin#2111
Rebased-From: 5d2e6e4
Cold-staking contacts are only saved when sending a delegation (with
label for staker address), but they cannot be added directly in the
addresses widget.
Remove the check preventing this.
Github-Pull: bitcoin#2121
Rebased-From: 058101f
Currently the confirmation pop-up is shown only when the RPC command
string is exactly "dumpwallet"/"dumpprivkey" (with no arguments).
Such command strings are not valid, as both RPC functions require one
argument (the filename for "dumpwallet" and the PIVX address for
"dumpprivkey").

So, when the user confirms the dialog, the RPC console returns an error
(as the argument is missing), showing the RPC help.

If the user, instead, enters the correct RPC strings (`dumpwallet
<filename>`/`dumpprivkey <pivxaddress>`), then **the warning dialog is
NOT presented** and the command is executed immediately.

Fix it by doing a pre-validation for the potentially dangerous commands.
Github-Pull: bitcoin#2122
Rebased-From: 1d63d1a
The for loop creates the required object directly

Github-Pull: bitcoin#2119
Rebased-From: ba9a699
Also drop the "ed" suffix from return error messages in these two
commands.

Github-Pull: bitcoin#2119
Rebased-From: c15c5c6
`initial_block_downloading` isn't a member of the `shield_pool_value`
object.

Github-Pull: bitcoin#2119
Rebased-From: b5404b5
…a diversified address different than the default one.

Github-Pull: bitcoin#2123
Rebased-From: 910b580
@furszy furszy self-assigned this Jan 5, 2021
@Fuzzbawls Fuzzbawls added this to the 5.0.0 milestone Jan 6, 2021
Final pull for v5.0.0 release

Github-Pull: bitcoin#2125
Rebased-From: b4db014
@furszy
Copy link
Author

furszy commented Jan 6, 2021

Updated with 2125 and 2114. Ready to go.

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 41fb982

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 41fb982

@Fuzzbawls Fuzzbawls merged commit f4df550 into PIVX-Project:5.0 Jan 6, 2021
@furszy furszy deleted the 5.0 branch May 27, 2023 01:56
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