Skip to content

remove enable stake pool as an option, always enabled#398

Merged
dajohi merged 6 commits intodecred:masterfrom
JoeGruffins:issue226-3
Jun 11, 2019
Merged

remove enable stake pool as an option, always enabled#398
dajohi merged 6 commits intodecred:masterfrom
JoeGruffins:issue226-3

Conversation

@JoeGruffins
Copy link
Copy Markdown
Member

@JoeGruffins JoeGruffins commented May 26, 2019

part of issue #226

remove ability to disable stakepoold communications.

After making these changes, the user will need to remove the "enablestakepoold=" option from their config or see an error during parsing

This will cause merge conflicts with another of my pull requests...

Comment thread config.go
@JoeGruffins
Copy link
Copy Markdown
Member Author

rebased and will not err if option is set, option is just ignored, description tells of deprecation

@dajohi
Copy link
Copy Markdown
Member

dajohi commented Jun 5, 2019

@JoeGruffins please rebase

@JoeGruffins
Copy link
Copy Markdown
Member Author

rebased

Comment thread config.go Outdated
Comment thread config.go
@dajohi dajohi merged commit f955590 into decred:master Jun 11, 2019
@JoeGruffins JoeGruffins deleted the issue226-3 branch August 6, 2019 12:03
girino added a commit to girino/dcrstakepool that referenced this pull request Sep 7, 2019
* commit 'b25f4307a2f1f956ab04c32919f7e88f66a4938d':
  multi: add ValidateAddress to stakepoold RPC (decred#406)
  show invalid ticket warning. (decred#395)
  multi: show last N voted tickets (decred#402)
  Get VoteVersion from stakepoold instead of dcrwallet (decred#403)
  remove enable stake pool as an option, always enabled (decred#398)
  prevent horizontal scrolling (decred#401)

# Conflicts:
#	config.go
girino added a commit to girino/dcrstakepool that referenced this pull request Sep 7, 2019
* commit 'f955590bef98437f79ee412969567f4d7ab117cf':
  remove enable stake pool as an option, always enabled (decred#398)

# Conflicts:
#	config.go
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.

4 participants