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] Implement generatetoaddress RPC command #2297

Merged

Conversation

furszy
Copy link

@furszy furszy commented Apr 6, 2021

Introducing a new RPC command generatetoaddress. Useful mostly for functional testing and a good number of updates that are coming from upstream.

@furszy furszy self-assigned this Apr 6, 2021
@furszy furszy added this to the 6.0.0 milestone Apr 11, 2021
random-zebra
random-zebra previously approved these changes Apr 17, 2021
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.

LGTM. Would be good to use this new RPC in some functional test, but we can add it later. utACK 16d95dd

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.

needs a small change to avoid parsing errors, and could fixup a couple styling nits while at it.

src/rpc/client.cpp Outdated Show resolved Hide resolved
src/rpc/mining.cpp Outdated Show resolved Hide resolved
src/rpc/mining.cpp Outdated Show resolved Hide resolved
@furszy
Copy link
Author

furszy commented May 11, 2021

Updated per review 👍

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.

This has conflicts with #2337.
But I'm ok if we want to give priority to this one, as it's straight forward (and it was submitted earlier 😉 )

src/rpc/mining.cpp Show resolved Hide resolved
src/rpc/mining.cpp Outdated Show resolved Hide resolved
@furszy furszy force-pushed the 2021_implement_generatetoaddress branch from d6df233 to 4fe3b03 Compare May 12, 2021 17:30
@furszy
Copy link
Author

furszy commented May 12, 2021

Updated per feedback.

@furszy furszy requested a review from Fuzzbawls May 12, 2021 20:57
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 4fe3b03

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 4fe3b03 and merging...

@random-zebra random-zebra merged commit f67f18d into PIVX-Project:master May 13, 2021
@random-zebra random-zebra added the Needs Release Notes Placeholder tag for anything needing mention in the "Notable Changes" section of release notes label May 13, 2021
@random-zebra random-zebra removed the Needs Release Notes Placeholder tag for anything needing mention in the "Notable Changes" section of release notes label Aug 24, 2021
@furszy furszy deleted the 2021_implement_generatetoaddress branch November 29, 2022 15:43
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