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

Generate rpc #1025

Closed
wants to merge 23 commits into from
Closed

Generate rpc #1025

wants to merge 23 commits into from

Conversation

Gnappuraz
Copy link
Member

At the moment the generate rpc relies still on the old bitcoin one, with some ugly workaround. This PR removes that completely, relying instead on the proposer logic of proposer.ccp which already handles the node's proposal mechanism.

@Gnappuraz Gnappuraz added the technical debt Cleaning up code which is there for historical reasons label Apr 29, 2019
@Gnappuraz Gnappuraz self-assigned this Apr 29, 2019
@Gnappuraz Gnappuraz added this to the 0.2 milestone Apr 29, 2019
Matteo Sumberaz added 23 commits April 30, 2019 15:33
Signed-off-by: Matteo Sumberaz <matteo@thirdhash.com>
Signed-off-by: Matteo Sumberaz <matteo@thirdhash.com>
Signed-off-by: Matteo Sumberaz <matteo@thirdhash.com>
Signed-off-by: Matteo Sumberaz <matteo@thirdhash.com>
Signed-off-by: Matteo Sumberaz <matteo@thirdhash.com>
Signed-off-by: Matteo Sumberaz <matteo@thirdhash.com>
Signed-off-by: Matteo Sumberaz <matteo@thirdhash.com>
Signed-off-by: Matteo Sumberaz <matteo@thirdhash.com>
Signed-off-by: Matteo Sumberaz <matteo@thirdhash.com>
Signed-off-by: Matteo Sumberaz <matteo@thirdhash.com>
The test does not call generate anymore cause generate does not create new addresses.

Signed-off-by: Matteo Sumberaz <matteo@thirdhash.com>
Signed-off-by: Matteo Sumberaz <matteo@thirdhash.com>
The generate call was in origin topping up the keypool, for this reason we moved to generatetoaddress.

Signed-off-by: Matteo Sumberaz <matteo@thirdhash.com>
Signed-off-by: Matteo Sumberaz <matteo@thirdhash.com>
Signed-off-by: Matteo Sumberaz <matteo@thirdhash.com>
Signed-off-by: Matteo Sumberaz <matteo@thirdhash.com>
Signed-off-by: Matteo Sumberaz <matteo@thirdhash.com>
Signed-off-by: Matteo Sumberaz <matteo@thirdhash.com>
Signed-off-by: Matteo Sumberaz <matteo@thirdhash.com>
Signed-off-by: Matteo Sumberaz <matteo@thirdhash.com>
Signed-off-by: Matteo Sumberaz <matteo@thirdhash.com>
Signed-off-by: Matteo Sumberaz <matteo@thirdhash.com>
Signed-off-by: Matteo Sumberaz <matteo@thirdhash.com>
@Gnappuraz
Copy link
Member Author

Closing if favor of #1051

@Gnappuraz Gnappuraz closed this May 3, 2019
@Gnappuraz Gnappuraz deleted the generate-rpc branch May 3, 2019 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
technical debt Cleaning up code which is there for historical reasons
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant