Skip to content

Remove v1 ticketbuyer in favor of v2 module. #1396

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

Merged
merged 1 commit into from
Feb 14, 2019
Merged

Conversation

jrick
Copy link
Member

@jrick jrick commented Feb 14, 2019

All v1-specific config options have been deprecated for several releases. These
options are no longer present after the module removal.

In addition to the removal of the v1 module, the startautobuyer and
stopautobuyer methods of the JSON-RPC server have been removed. This is due to
the v2 buyer being controlled exclusively by the main package when
--enableticketbuyer is set and because these RPCs contained many v1-specific
options while failing to specify the account to purchase from. Multiple v2
ticketbuyer instances are still possible control through the gRPC API.

@jrick jrick force-pushed the removetbv1 branch 3 times, most recently from ba444d6 to d7c593f Compare February 14, 2019 17:09
All v1-specific config options have been deprecated for several releases.  These
options are no longer present after the module removal.

In addition to the removal of the v1 module, the startautobuyer and
stopautobuyer methods of the JSON-RPC server have been removed.  This is due to
the v2 buyer being controlled exclusively by the main package when
--enableticketbuyer is set and because these RPCs contained many v1-specific
options while failing to specify the account to purchase from.  Multiple v2
ticketbuyer instances are still possible control through the gRPC API.
Copy link
Member

@davecgh davecgh left a comment

Choose a reason for hiding this comment

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

The code reads correctly, and I tested it with local a simnet setup as well. I also confirmed the updated module details.

@jrick jrick merged commit 3a69079 into decred:master Feb 14, 2019
@jrick jrick deleted the removetbv1 branch February 14, 2019 20:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants