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

multi: Configurable account for paying dividends #191

Merged
merged 2 commits into from
Apr 22, 2020

Conversation

jholdstock
Copy link
Member

Add a config item to select the account used to pay reward dividends. Previously hard coded to account 0 (default).

config.go Outdated
@@ -95,7 +96,8 @@ type config struct {
MaxGenTime time.Duration `long:"maxgentime" ini-name:"maxgentime" description:"The share creation target time for the pool. Valid time units are {s,m,h}. Minimum 2 seconds. This currently should be below 30 seconds to increase the likelihood a work submission for clients between new work distributions by the pool."`
PaymentMethod string `long:"paymentmethod" ini-name:"paymentmethod" description:"The payment method of the pool. {pps, pplns}"`
LastNPeriod time.Duration `long:"lastnperiod" ini-name:"lastnperiod" description:"The time period of interest when using PPLNS payment scheme. Valid time units are {s,m,h}. Minimum 60 seconds."`
WalletPass string `long:"walletpass" ini-name:"walletpass" description:"The wallet passphrase."`
WalletPass string `long:"walletpass" ini-name:"walletpass" description:"The wallet passphrase to use when paying dividends to pool contributors."`
WalletAccount uint32 `long:"walletaccount" ini-name:"walletaccount" description:"The wallet account to use when paying dividends to pool contributors."`
Copy link
Member

@dnldd dnldd Apr 21, 2020

Choose a reason for hiding this comment

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

The description should read "The wallet account that will receive mining rewards when not mining as a solo pool".

Copy link
Member

@dnldd dnldd left a comment

Choose a reason for hiding this comment

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

Looks good except for the description update.

@jholdstock jholdstock merged commit f2c3e8f into decred:master Apr 22, 2020
@jholdstock jholdstock deleted the conf-acc branch October 6, 2020 09:22
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.

2 participants