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

[RPC] Shield address setlabel/getaddressesbylabel #2609

Merged
merged 1 commit into from
Nov 19, 2021

Conversation

Fuzzbawls
Copy link
Collaborator

This adds shield address support to the setlabel and
getaddressesbylabel RPC commands. setlabel now accepts and validates
shield address input argument, where getaddressesbylabel now returns
shield addresses with a label matching the input argument.

Companion to #2600 that allows CLI users to set a label for shield addresses
after-the-fact, as well as see shield addresses with an assigned label in the
response of getaddressesbylabel

Also added relevant release notes for the changes in #2600.

Regression tests added as well to cover the setlabel command.

@Fuzzbawls Fuzzbawls added the RPC label Oct 19, 2021
@Fuzzbawls Fuzzbawls self-assigned this Oct 19, 2021
@Fuzzbawls Fuzzbawls added this to the 5.4.0 milestone Oct 27, 2021
This adds shield address support to the `setlabel` and
`getaddressesbylabel` RPC commands. `setlabel` now accepts and validates
 shield address input argument, where `getaddressesbylabel` now returns
 shield addresses with a label matching the input argument.
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 db68b90

Copy link

@furszy furszy left a comment

Choose a reason for hiding this comment

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

code ACK db68b90, merging..

@furszy furszy merged commit ce94ee6 into PIVX-Project:master Nov 19, 2021
This pull request was closed.
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