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

Remove support for "0" as an alternative to "" when the default is requested #2622

Merged
merged 2 commits into from
Jan 12, 2019

Conversation

codablock
Copy link

See discussion: #2617 (comment)

Might need a backport to v13

src/rpc/rpcevo.cpp Outdated Show resolved Hide resolved
@UdjinM6 UdjinM6 added this to the 14.0 milestone Jan 11, 2019
Co-Authored-By: codablock <ablock84@gmail.com>
Copy link

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

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

utACK

UdjinM6
UdjinM6 approved these changes Jan 12, 2019
@UdjinM6
Copy link

UdjinM6 commented Jan 12, 2019

Not sure how I managed to do it twice in a row 🙈

@codablock codablock merged commit fed4716 into dashpay:develop Jan 12, 2019
@codablock codablock deleted the pr_dip3_protx_no0default branch January 12, 2019 13:17
codablock added a commit to codablock/dash that referenced this pull request Jan 12, 2019
…quested (dashpay#2622)

* Remove support for "0" as an alternative to "" when the default is requested

* Update src/rpc/rpcevo.cpp

Co-Authored-By: codablock <ablock84@gmail.com>
codablock added a commit that referenced this pull request Jan 14, 2019
…quested (#2622) (#2624)

* Remove support for "0" as an alternative to "" when the default is requested

* Update src/rpc/rpcevo.cpp

Co-Authored-By: codablock <ablock84@gmail.com>
@UdjinM6 UdjinM6 added the RPC Some notable changes to RPC params/behaviour/descriptions label Mar 5, 2019
thephez added a commit to thephez/dash-docs that referenced this pull request Apr 10, 2019
 - Remove support for "0"

Related to dashpay/dash#2622
thephez added a commit to dash-docs/dash-docs that referenced this pull request Apr 10, 2019
* RPC - Update masternode current and winner
 - Remove protocol, lastseen, activeseconds

* RPC - Remove gobject getvotes
 - Related to dashpay/dash#2815

* RPC - Protx diff update
 - Add merkleRootQuorums field

Related to dashpay/dash#2833

* RPC - listunspent update

Related to dashpay/dash#2727

* RPC - protx update_registrar update
 - Rename votingKeyAddr to votingAddress

Related to dashpay/dash#2747

* RPC - Add bls fromsecret

Related to dashpay/dash#2841

* RPC - Start adding quorum RPCs

* RPC - Add quorum info details Minor formatting updates

* RPC - Add placeholders for remaining quorum sub-commands

* RPC - Add quorum sign/hasrecsig/isconflicting details

* RPC - Add note to regtest only quorum RPCs

* RPC - Add more detail to quorum dkgstatus

* Typo fix

* Minor format and example updates

* RPC - Add more quorum dkgstatus details
 - Field descriptions

* RPC - Add quorum dkgstatus detail level 1 and 2 info

* RPC - protx update
 - Remove support for "0"

Related to dashpay/dash#2622
thephez added a commit to thephez/dash-docs that referenced this pull request May 17, 2019
* RPC - Update masternode current and winner
 - Remove protocol, lastseen, activeseconds

* RPC - Remove gobject getvotes
 - Related to dashpay/dash#2815

* RPC - Protx diff update
 - Add merkleRootQuorums field

Related to dashpay/dash#2833

* RPC - listunspent update

Related to dashpay/dash#2727

* RPC - protx update_registrar update
 - Rename votingKeyAddr to votingAddress

Related to dashpay/dash#2747

* RPC - Add bls fromsecret

Related to dashpay/dash#2841

* RPC - Start adding quorum RPCs

* RPC - Add quorum info details Minor formatting updates

* RPC - Add placeholders for remaining quorum sub-commands

* RPC - Add quorum sign/hasrecsig/isconflicting details

* RPC - Add note to regtest only quorum RPCs

* RPC - Add more detail to quorum dkgstatus

* Typo fix

* Minor format and example updates

* RPC - Add more quorum dkgstatus details
 - Field descriptions

* RPC - Add quorum dkgstatus detail level 1 and 2 info

* RPC - protx update
 - Remove support for "0"

Related to dashpay/dash#2622
thephez added a commit to dash-docs/dash-docs that referenced this pull request May 17, 2019
* RPC - Update masternode current and winner
 - Remove protocol, lastseen, activeseconds

* RPC - Remove gobject getvotes
 - Related to dashpay/dash#2815

* RPC - Protx diff update
 - Add merkleRootQuorums field

Related to dashpay/dash#2833

* RPC - listunspent update

Related to dashpay/dash#2727

* RPC - protx update_registrar update
 - Rename votingKeyAddr to votingAddress

Related to dashpay/dash#2747

* RPC - Add bls fromsecret

Related to dashpay/dash#2841

* RPC - Start adding quorum RPCs

* RPC - Add quorum info details Minor formatting updates

* RPC - Add placeholders for remaining quorum sub-commands

* RPC - Add quorum sign/hasrecsig/isconflicting details

* RPC - Add note to regtest only quorum RPCs

* RPC - Add more detail to quorum dkgstatus

* Typo fix

* Minor format and example updates

* RPC - Add more quorum dkgstatus details
 - Field descriptions

* RPC - Add quorum dkgstatus detail level 1 and 2 info

* RPC - protx update
 - Remove support for "0"

Related to dashpay/dash#2622
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RPC Some notable changes to RPC params/behaviour/descriptions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants