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

POW Mining Doc update #1134

Merged
merged 4 commits into from
Oct 16, 2020
Merged

POW Mining Doc update #1134

merged 4 commits into from
Oct 16, 2020

Conversation

hashbender
Copy link
Contributor

This PR updates and refreshes the POW mining documentation.

We've included some screenshots which help demonstrate the configuration page.

There is also a section which includes the current generation of ASIC miners

@hashbender
Copy link
Contributor Author

@jholdstock any update on this?

Copy link
Member

@jholdstock jholdstock left a comment

Choose a reason for hiding this comment

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

Sorry for the delay on reviewing this, we've been pushing to get the 1.6 release out and documentation has taken a bit of a back seat.

Overall this is looking good, thanks a lot for the contribution. I've dropped a few small suggestions but nothing major to fix.

Please update rig_config.png so it does not include a mainnet DCR address. Leaving it blank or using a testnet address is fine.

docs/mining/overview.md Outdated Show resolved Hide resolved
docs/mining/overview.md Outdated Show resolved Hide resolved
docs/mining/overview.md Outdated Show resolved Hide resolved
docs/mining/overview.md Outdated Show resolved Hide resolved
docs/mining/overview.md Show resolved Hide resolved
docs/mining/overview.md Outdated Show resolved Hide resolved
docs/mining/overview.md Outdated Show resolved Hide resolved
docs/mining/overview.md Outdated Show resolved Hide resolved
docs/mining/overview.md Outdated Show resolved Hide resolved
docs/mining/overview.md Outdated Show resolved Hide resolved
@jholdstock
Copy link
Member

Thanks for the work @GuzmanPintos & @nitronick600!

@jholdstock jholdstock merged commit 8e40661 into decred:master Oct 16, 2020
@GuzmanPintos
Copy link

Hey @jholdstock, when are these changes getting deployed? 😄

@jholdstock
Copy link
Member

@GuzmanPintos
Copy link

Thanks!

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.

3 participants