-
Notifications
You must be signed in to change notification settings - Fork 27
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
Clean up pool to solo guide #376
base: master
Are you sure you want to change the base?
Conversation
I plan to squash commits before this is merged. |
Can you please name your commits? |
I plan to squash them later. |
fafd6ef
to
037653f
Compare
Glad to see some more work is being done on the site :). Re: account managers, I would suggest leaving these out of the solo guide. There should be one, simple default way to go through the guide with any other complexity "branched off" for users who need it. BAM is great, but it's one more complex thing for a user to setup and it has plenty of quirks which make it difficult to use, so I don't think it should be the default method. We already have this noted on the first page of the solo crunching guide: Note: If you are an existing BOINC user and use an account manager such as BAM, you can add projects via your account manager so long as accounts at the projects all use the same e-mail address (BAM's default). I do think some design changes in terms of layout and presentation could make a drastic difference to readability. This is beyond my HTML/CSS knowledge, but for example, why do we need the BOINC videos to be a different column here: https://gridcoin.us/guides/boinc-install.htm. Breaking up text visually could be a major help. |
481e944
to
dd7fc79
Compare
I think design changes is a bit out of scope for this PR. |
I also just submitted a handful of very small PRs for usability you may want to check out and/or incorporate into yours. |
Can we finish this please? |
Are there any changes required? |
Add split cpid instructions Add notice about lack of rewards with pool account Remove account manager info in line with solo guide Expand split cpid warning
dd7fc79
to
f2c68ea
Compare
Closes #357
Todo: